Adjusted validation regex #34
Conversation
To relax AMP domain validation, the validation regex was adjusted to allow for adding domains like amp.localhost, amp.sub.domain.com or amp.com/kentico (to support virtual directories). The new regex is taken from https://mathiasbynens.be/demo/url-regex, specifically the one from @diegoperini, as it has the best results in validation.
|
Hello @kentico-timothyf, could you please take a look at the regular expression if it is OK for you? |
petrsvihlik
requested review from
petrsvihlik
and removed request for
kentico-timothyf
|
Since the new version of Kentico EMS is being prepared for release. We are going to groom the issue after the release - the 27th of November. |
|
Hey @RichardKalinec, we would like to thank you for contributing to Kentico's open-source project. As a sign of our appreciation, we would like to send you some Kentico Developer Community SWAG! Please fill out your postal address if you are interested. This comment was automatically generated. If you've submitted more than one pull request, it's OK to fill out the form just once. Kentico Developer Community If there’s anything we can do to help, please don’t hesitate to reach out to us at developerscommunity@kentico.com |
To relax AMP domain validation, the validation regex was adjusted to allow for adding domains like amp.localhost, amp.sub.domain.com or amp.com/kentico (to support virtual directories). The new regex is taken from https://mathiasbynens.be/demo/url-regex, specifically the one from @diegoperini, as it has the best results in validation.
Fixes #4
EDIT: Related also to #6