- Issue created by @AstonVictor
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 9:50am 1 December 2023 - 🇺🇦Ukraine AstonVictor
Prepared a new MR.
Created a new config form to enable redirects per bundle. The URL of the config form is
/admin/config/search/bundle-redirect
.
Also, created a menu link to the config form, added schema config for settings and implementedhook_update()
to enable the config for all bundles in order not to break the current state. - Status changed to Closed: won't fix
about 1 year ago 11:52am 1 December 2023 - 🇮🇳India Akhil Babu Chengannur
I have tested this. The patch functions correctly, but it does not enable/disable the 'URL Redirects' section on the node edit page. This is because the 'redirect' module, which is a dependency of this module, also adds the same 'URL Redirects' section through its own form alter.
https://git.drupalcode.org/project/redirect/-/blob/8.x-1.x/redirect.modu...
I believe this should be incorporated as a feature in the 'redirect' module and not in this module. - Status changed to Needs review
about 1 year ago 12:26pm 1 December 2023 - 🇺🇦Ukraine AstonVictor
Hi @Akhil Babu,
I guess you are not familiar with the module. It adds an additional section on the node edit page.
So, there are two redirect sections: one from the redirect module (add a redirect from the node to any other pages) and another from the current module (add a redirect from other pages to the node).The task is to add access control to the section from the current module.
- Status changed to RTBC
12 months ago 11:30am 22 December 2023 - 🇮🇳India Anjali Mehta
Hi @Aston Victor,
I have reviewed the changes, and they look great. The addition of the new section with redirect option on node edit page is successful. marking this as RTBC.
Thank you!
- Status changed to Fixed
12 months ago 11:49am 22 December 2023 - 🇺🇦Ukraine AstonVictor
thanks for the review.
The MR is merged and changes will be added in the next release.
Automatically closed - issue fixed for 2 weeks with no activity.