Problem/Motivation

It is not possible to skip phone verification for a specific role (for example) or override the code verification value.

Proposed resolution

Add a dispatcher to manage the verification process.

Remaining tasks

This fix will useful in related issues.

https://www.drupal.org/project/smsframework/issues/3142993
https://www.drupal.org/project/smsframework/issues/3143130

User interface changes, API changes and Data model changes

There is no change.

CommentFileSizeAuthor
#7 3199970-phone-number-verification-7.patch12.02 KBabramm
#6 3199970-interdiff-2.txt385 bytesspheresh
#6 3199970-phone-number-verification-3.patch11.97 KBspheresh
#4 3199970-interdiff-1.txt1.82 KBspheresh
#4 3199970-phone-number-verification-2.patch11.97 KBspheresh
#2 3199970-phone-number-verification.patch11.12 KBspheresh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

spheresh created an issue. See original summary.

spheresh’s picture

spheresh CreditAttribution: spheresh commented
Status: Active » Needs review
FileSize
3199970-phone-number-verification.patch11.12 KB

Status: Needs review » Needs work

The last submitted patch, 2: 3199970-phone-number-verification.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

spheresh’s picture

spheresh CreditAttribution: spheresh commented
FileSize
3199970-phone-number-verification-2.patch11.97 KB
3199970-interdiff-1.txt1.82 KB

with testing fixes

spheresh’s picture

spheresh CreditAttribution: spheresh commented
Status: Needs work » Needs review
spheresh’s picture

spheresh CreditAttribution: spheresh commented
FileSize
3199970-phone-number-verification-3.patch11.97 KB
3199970-interdiff-2.txt385 bytes

with typo fix ...

abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
CreditAttribution: abramm at DevBranch, Drupal Ukraine Community commented
FileSize
3199970-phone-number-verification-7.patch12.02 KB

Here's a patch for 2.x.

NOTE: I didn't test it, just re-rolled.

ivnish’s picture

ivnish CreditAttribution: ivnish commented
Status: Needs review » Reviewed & tested by the community

Patch #7 works as expected