- Issue created by @krystalcode
- Status changed to Needs review
18 days ago 1:44pm 11 August 2025 - First commit to issue fork.
- π¬π§United Kingdom alexpott πͺπΊπ
@krystalcode this is a really neat idea. +1 for moving validation out of forms and into constraints. I'm going to add some specific test coverage of constraint and check that we have and add if necessary coverage for the form.
I've open an MR and merged this patch to do this.
- Merge request !159Issue #3525919: Validate hash using entity constraint instead of form validation β (Merged) created by alexpott
- π¬π§United Kingdom alexpott πͺπΊπ
Added unit test and paired the patch back to the minimum necessary change.
I think this is a nice change that improves redirect validation and the ability to create redirects via an API.
- π¬π§United Kingdom alexpott πͺπΊπ
I've addressed @berdir's feedback - thanks for the review!
-
berdir β
committed 22ce0d70 on 8.x-1.x authored by
alexpott β
Issue #3525919 by alexpott, krystalcode: Validate hash using entity...
-
berdir β
committed 22ce0d70 on 8.x-1.x authored by
alexpott β
- π¨πSwitzerland berdir Switzerland
Looks good now, simpler and fewer unrelated changes.