- Issue created by @anilkumar.kollavajjala
- @sourav_paul opened merge request.
- ๐ฎ๐ณIndia sourav_paul Kolkata
I created a field in the config form to customize the error message.
please review..
- ๐บ๐ธUnited States tr Cascadia
Changes to configuration values need a hook_update_N() and a test case for the update hook.
- ๐บ๐ธUnited States tr Cascadia
cspell, phpcs, phpstan and phpunit failures need to be addressed.
The update hook should probably be 10200 at this point - the 8xxx series isn't meant to continue indefinitely. 10 for the Major version of Drupal and 2 for the Major version of Honeypot.
I would prefer that the update test subclass UpdatePathTestBase to ensure that all the steps are done. - Status changed to Needs work
20 days ago 6:04pm 4 February 2025 - ๐ฌ๐งUnited Kingdom joehuggans Harrogate, UK
joe huggans โ made their first commit to this issueโs fork.
- ๐ฌ๐งUnited Kingdom joehuggans Harrogate, UK
Getting some failures with the HoneypotFormCacheTest, not able to replicate the failure locally, not much of an idea currently what could be causing it.
- ๐ฌ๐งUnited Kingdom joehuggans Harrogate, UK
Do we need to think about people who might update the module but forget to run the database updates? So check that $this->config->get('time_limit_message') isn't NULL before setErrorByName?