- Issue created by @baldwinlouie
- Merge request !44Added function to validate the end_user fields β (Merged) created by baldwinlouie
- Status changed to Needs review
10 months ago 7:28pm 24 January 2024 - πΊπΈUnited States baldwinlouie
@yas, please review this patch. I've added validation to make sure the user does not choose values that are the same as the
drupal_enduser_email
anddrupal_enduser_username
- Status changed to RTBC
10 months ago 9:04pm 24 January 2024 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the fix. I posted my comment to use a guard clause.
Thanks
- Status changed to Needs work
10 months ago 9:04pm 24 January 2024 - Status changed to Needs review
10 months ago 9:35pm 24 January 2024 - Status changed to Needs work
10 months ago 9:53pm 24 January 2024 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the update. Can you please take a look at phpcs errors at https://git.drupalcode.org/project/facade/-/jobs/687494? Thanks!
- Status changed to Needs review
10 months ago 11:06pm 24 January 2024 - Status changed to Needs work
10 months ago 11:38pm 24 January 2024 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the update. I posted my comment. Thanks!
- Status changed to Needs review
10 months ago 12:01am 25 January 2024 - Status changed to RTBC
10 months ago 3:33am 25 January 2024 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the update. It looks good to me now. I'll merge the patch to
1.0.x
, and close this issue as Fixed. -
yas β
committed 7ee43322 on 1.0.x authored by
baldwinlouie β
Issue #3416786 by baldwinlouie, yas: Add a validation to cheak a tenant...
-
yas β
committed 7ee43322 on 1.0.x authored by
baldwinlouie β
- Status changed to Fixed
10 months ago 3:37am 25 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.