- Status changed to Needs review
about 1 year ago 7:45am 5 October 2023 - last update
about 1 year ago 11 pass - 🇮🇳India pradhumanjainOSL
I made changes as per comment #12.
Please review. - 🇫🇷France simon georges Rouen, France
The previous patch cannot work, considering it will add a "/" on external urls before testing them for externality, so the patch in #5 is the correct one, and still applies cleanly on current version.
- 🇫🇷France simon georges Rouen, France
(in our case, we wanted to add a custom class on the link to be able to style it specifically)
- last update
about 1 year ago 11 pass - 🇮🇳India prashant.c Dharamshala
Created MR from the patch provided by #5 🐛 You are not allowed to specify an external URL together with internal:/ Needs review . Could not find any issues while testing locally.
Thanks!
- last update
12 months ago 11 pass - Status changed to Fixed
12 months ago 9:14am 5 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.