- Issue created by @scott_euser
- π¬π§United Kingdom scott_euser
Would welcome feedback on the proposed steps.
- Assigned to scott_euser
- Status changed to Needs work
over 1 year ago 4:29am 24 October 2023 - π¬π§United Kingdom scott_euser
Am thinking this probably needs a
8.x-2.x
branch in case anyone is extending/overriding services from this module, its probably unavoidable to cause breaks there. For those not extending/overriding services, should not be a problem to have a smooth upgrade path maintaining the status quo. - π¬π§United Kingdom scott_euser
Progress is happening in the MR, crossing things off as I get through them.
- @scott_euser opened merge request.
- π¬π§United Kingdom scott_euser
Adding credit to bobi-mel for type hints from https://www.drupal.org/project/site_settings/issues/3176342 β¨ Add type hint in function parameters and return values Fixed - thanks!
- π¬π§United Kingdom scott_euser
Nearly there with this. Need to resolve existing test failures, add new tests to cover new functionality, and update documentation. Then follow-up with this probably https://www.drupal.org/project/site_settings/issues/3398584 β¨ Remove _entity from site settings entity type IDs Postponed
Would appreciate feedback on approach and suggestions before we eventually lock in to this direction of travel.
- π¬π§United Kingdom scott_euser
Okay tests are all passing now, including new tests for twig extension for the full loader.
One warning from phpstan which is ignored in core; support request raised here for help ignoring that in contrib as well: https://www.drupal.org/project/drupal/issues/3399282 π¬ Document how to use phpstan.neon in a contrib module Active
Just the documentation updates to be made otherwise this is ready now for 2.0.x branch.
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 6:08am 6 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.