Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(fix) Bug Report: Inconsistency in User Verification Status between E… #1450

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

divijshrivastava
Copy link

…mail and Phone Number #1392

What does this PR do?

This fixes Issue #1392 dealing with incorrect notification when verifying phone number of user.

Test Plan

Following steps have been followed to check if this works right:

  1. Login -> Go to dashboard -> Project -> Auth -> Select User
image
  1. In Overview tab click on Verify Account
  2. Click on Verify Phone
image
  1. Checked the Notification that says : {User} has been verified.
image
  1. Click on Unverify Phone
  2. Check the Notification that says: {User} has been unverified.
image

Related PRs and Issues

This resolves issue #1392

Have you read the Contributing Guidelines on issues?

Yes

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Oct 24, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-10-24 22:19:34 CET

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Oct 24, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-10-24 22:19:41 CET

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Oct 24, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-10-24 22:19:42 CET

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Oct 24, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-10-24 22:19:42 CET

@divijshrivastava
Copy link
Author

@stnguyen90 could you please review this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant