-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EZP-32157: Added option to choose SA when creating custom alias #1583
Conversation
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this added via listener instead directly to a form? Don't know much about AdminUI architecture...
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
@alongosz I guess in order to keep it clean. In this form type there are 4 other listeners working exactly the same. |
src/bundle/Resources/views/content/tab/url/modal_add_custom_url.html.twig
Outdated
Show resolved
Hide resolved
src/bundle/Resources/views/content/tab/url/modal_add_custom_url.html.twig
Outdated
Show resolved
Hide resolved
src/bundle/Resources/views/content/tab/url/modal_add_custom_url.html.twig
Outdated
Show resolved
Hide resolved
Co-authored-by: DominikaK <dominika.kurek@ez.no>
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Form/EventListener/AddSiteAccessFieldBasedOnContentListener.php
Outdated
Show resolved
Hide resolved
src/lib/Tests/Form/Data/Content/CustomUrl/CustomUrlAddDataTest.php
Outdated
Show resolved
Hide resolved
ping @ezsystems/documentation-team I think this functionality will benefit from some minor description somewhere (in doc or RN). |
src/bundle/Resources/public/js/scripts/admin.location.add.custom_url.js
Outdated
Show resolved
Hide resolved
Co-authored-by: Dariusz Szut <dew326@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on eZ Platform EE 2.5 with branch.
Moving to Doc to cover the different combinations.
Could yuo please merge up changes @barw4 ? |
There are issues with a multi repository setup. If you have the same location ids in both repositories the siteaccesschoicer will display all possible siteaccess even if they are stored in the other repository |
Choice of Site Access has been added to the custom URL Alias form to help editors with providing a valid path when using
Place at site root
option when certain Site Access has parametercontent.tree_root.location_id
defined.Checklist:
$ composer fix-cs
)