- π§πͺBelgium al0a
Rerolled the patch of #24 so it applies on the 4.0.x version.
- πΊπΈUnited States loze Los Angeles
Applied the patch to 4.x and it appears to be working as advertised.
One suggestion, I think it would be better if instead of a checkbox to enable on all forms it was radio buttons
[] Add to all forms except those listed
[] Add to only the forms listedIn my case there are only a few forms I want to disable it on, so listing every possible form I want it on for would be difficult, plus I don't want to have to update the list when new forms are added to the site.
- π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
Copying NW tasks from #21 to ID.
- πΊπΈUnited States edmund.dunn Olympia, WA
I rerolled this to move the menu link from config->system to config->user interface. Also had to move the setting.yml file into the config/install directory to prevent an error when installing the module.
This works well for our needs.
- Status changed to Needs review
11 months ago 5:21pm 26 April 2024 - last update
11 months ago 1 pass - π§πͺBelgium ludo.r Brussels
Patch #31 seems to work on our side (Using version 4.1.0).
I also agree with suggestion in #28, to have both options (white/black-list).
- First commit to issue fork.
- πΊπΈUnited States skyriter
I created a separate issue specific to exceptions, building off of the the most recent work done for this issue: https://www.drupal.org/project/clientside_validation/issues/3495738#comm... β¨ Add a configuration option for exceptions to all forms selection Active
- πΊπΈUnited States skyriter
I should note that we have tested this patch and it appears to be working without issue.
- Merge request !21Issue #2949540: Adds patch changes to allow specific forms β (Open) created by skyriter
- πΊπΈUnited States skyriter
I have added the patch file, but even though this is working on my local and in our testing environment, I'm getting a PHPUnit error that I do not understand. If anyone could help with this, I'd appreciate it.
- πΊπΈUnited States skyriter
Now that I've updated the config that gets installed, the tests are passing.
- πΊπΈUnited States skyriter
skyriter β changed the visibility of the branch 3495738-form-exceptions to hidden.
- πΊπΈUnited States skyriter
Adding the newest patch, after the issues identified by coderdan β were resolved.
This patch removes any mention of webform.
- πΊπΈUnited States j-barnes
Updated the latest patch to include a small change to allow for patterns similar to how config ignore works. We encountered a use case where enabling client-side validation for all webform submissions was necessary. However, manually adding hundreds of form IDs was impractical.
Example:
webform_submission_name_change_test_form webform_submission_*