- Issue created by @darrenwh
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 11:06pm 4 February 2023 - 🇮🇳India Nayana Ramakrishnan
Verified the patch #2 and tested it on Drupal version 9.5.x and User email verification version 8.x-1.x. The patch works fine but there was a warning message wile applying the patch:
user_email_verification-typo-was-has-3338980-2.patch:10: trailing whitespace. $this->messenger()->addError($this->t('Your verification link has expired. Request a new one using the Checking patch src/Controller/UserEmailVerificationVerify.php... Applied patch src/Controller/UserEmailVerificationVerify.php cleanly. warning: 1 line adds whitespace errors.I have added the before and after patch screenshots for reference.
- Status changed to Needs work
over 1 year ago 8:06am 6 February 2023 - 🇮🇳India Sonal Gyanani
Sonal Gyanani → made their first commit to this issue’s fork.
- @sonal-gyanani opened merge request.
- Status changed to Needs review
over 1 year ago 9:35am 6 February 2023 - 🇮🇳India Nayana Ramakrishnan
Verified the changes of MR!2 and tested it on Drupal version 9.5.x and User email verification version 8.x-1.x. The changes were applied cleanly without any warning message. The screenshots are same as in #5 🐛 Typo 'was' should read 'has' Fixed . RTBC
-
darrenwh →
authored 1156ce82 on 8.x-1.x
Issue #3338980 by Sonal Gyanani, darrenwh, nayana_mvr, apaderno: Typo '...
-
darrenwh →
authored 1156ce82 on 8.x-1.x
- Status changed to Fixed
12 months ago 1:18pm 6 July 2023 - 🇩🇪Germany Antonnavi Berlin
Hello all!
Thank you for help! Changes were committed to the development branch.
Automatically closed - issue fixed for 2 weeks with no activity.