- Issue created by @rteijeiro
- First commit to issue fork.
- Merge request !159Issue #3528054 by almunnings, rteijeiro, dydave: Fixed regression introduced... → (Open) created by dydave
- 🇫🇷France dydave
Thanks a lot Ruben (@rteijeiro) for your help on this issue, it's greatly appreciated! 🙂
The changes suggested in the patch in the issue summary have been added to the merge request MR !159 👌
To be honest, I did not test this directly myself, since I'm not really using the
admin_toolbar_links_access_filtermodule anymore...I think it was marked
deprecatedand supposed to be removed when future releases drop support for core versions below 10.3.
So personally, I wouldn't really be interested in working on this, but I would not be opposed to getting this added to the module.If you could please help testing and reviewing the changes in the MR and if you think they could provide an acceptable solution, then I would be glad to get these changes merged in.
Again, since we're looking at removing this module, we wouldn't be bothering with Tests, docs and such. But if getting this patch committed could help with your projects, we would certainly be glad to do so 🙂
We would greatly appreciate if you could please try testing merge request MR !159 and give us your feedback or reviews.
Feel free to let me know if you spot anything or if I missed anything else in the comments above, I would certainly be glad to help.
Thanks in advance!