Thanks for moving this along @wesleymusgrove. Nice rework of the JS.
I've closed the other MR, so there's no confusion.
I've done a quick test with the latest changes and seems to be working fine.
Hi @wesleymusgrove
Thanks for your comments, I've implemented them with some minor changes.
Also changed to javascript functions to use the enterprise version based on the use_enterprise setting
Started a merge request that rerolls the last patch to the last version of the reCAPTCHA module and adds some small improvements.
Have not had the time to go over the comments/concerns of @wesleymusgrove
jonjacobs → made their first commit to this issue’s fork.
jonjacobs → created an issue.
Needed this also, I've worked a little bit more on the previous patch, with the following changes:
- For some situations where paragraphs were translated, all existing translations were overwritten, should now be fixed
- Changed how the calls to chatgpt are made, instead of 1 big call (which resulted in a lot of tokens being used + keys being overwritten) the values are now being loaded separately
- Kept all existing paragraph fields instead of only the translatable ones
Probably needs some more refining and cleaning up.
JonJacobs → created an issue.
Tested with the latest release (1.2.2), can confirm that it is fixed.
Hi Tim
You are correct, module was updated from 1.1.1 to 1.2.0 in the process.
JonJacobs → created an issue.
JonJacobs → created an issue.