- Issue created by @Chandreshgiri Gauswami
- Status changed to Needs review
11 months ago 3:10pm 4 January 2024 - last update
11 months ago 4 pass - Issue was unassigned.
- 🇮🇳India Sandeep_k New Delhi
@Chandreshgiri Gauswami, Verified and tested patch 3412344-2.patch on Drupal version- 10.1.8-dev. The patch was applied successfully and looks good to me.
Testing Steps:- Enable recaptcha and its dependent modules.
- Go to /admin/config/people/captcha/recaptcha- to reproduce this. Attached before result.
- Download the shared patch and Apply.
- Go to /admin/config/people/captcha/recaptcha- Reverify this issue.
Testing Results:
A new Email field is added there for user reference. RTBC++ - Status changed to RTBC
11 months ago 7:54am 5 January 2024 - 🇩🇪Germany Anybody Porta Westfalica
@Liam Morland I can't really see a benefit here for the majority. I don't think we should put that into the module, what do you think?
Voting for closed won't fix.
- 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
It could be useful. I haven't really worked on this module since 7.x-1.x. I don't think the update hook is needed. The value should be null when empty.