- Issue created by @chandreshgiri gauswami
- Status changed to Needs review
over 1 year ago 3:10pm 4 January 2024 - last update
over 1 year 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
over 1 year 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.
- Status changed to Needs work
4 months ago 10:24am 16 December 2024 - 🇩🇪Germany Anybody Porta Westfalica
Please prepare a MR to proceed here. We're not using patches anymore. Thanks!
- First commit to issue fork.
- 🇮🇳India atul_ghate
atul_ghate → changed the visibility of the branch 3412344-email-address-field to hidden.
- 🇮🇳India atul_ghate
atul_ghate → changed the visibility of the branch 3412344-email-address-field to active.
- 🇮🇳India atul_ghate
I have converted patch #2 into a Merge Request. Please review it.
- 🇮🇳India divyansh.gupta Jaipur
I have reproduced the issue and MR-36 successfully applied and the fix is working fine for me.
Before:
After:
Thus moving this to RTBC+!