- Issue created by @dieterholvoet
- Status changed to Needs review
over 1 year ago 1:20pm 25 May 2023 - πΊπΈUnited States arrow Chicago
This seems to work as expected. One thing I noticed is that the description text for the setting requires the entity type to exist. When creating the new entity, the machine name does not yet exist so it is not populated.
I would also like to see someone test upgrading from the existing behavior to ensure that existing URLs for entities are maintained after the upgrade and that disabling URLs on those entities works as expected.
- Status changed to Needs work
over 1 year ago 9:21pm 22 June 2023 - π§πͺBelgium dieterholvoet Brussels
I removed the 'at
' portion of the field description. You're raising a good point about the upgrade path, we should probably add an update hook that sets the value for existing entity types. Setting back to Needs work. - Status changed to Needs review
over 1 year ago 2:01pm 3 August 2023 - π§πͺBelgium dieterholvoet Brussels
I added an update hook that sets the
standalone_url
option to true for existing entity types, to make sure existing behaviour is preserved. -
dieterholvoet β
committed c834a714 on 2.x
Issue #3362626 by dieterholvoet, arrow: Make standalone url optional
-
dieterholvoet β
committed c834a714 on 2.x
- Status changed to Fixed
3 days ago 9:27am 19 December 2024