Skip to content

fix: correct phone verification success notification - #3164

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/phone-verification-notification-message
Aug 17, 2026
Merged

fix: correct phone verification success notification#3164
ChiragAgg5k merged 1 commit into
mainfrom
fix/phone-verification-notification-message

Conversation

@cursor

@cursor cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

Summary

After updating phone verification and invalidating the user store, the success toast used the new phoneVerification value with inverted copy, so Verify showed "unverified" and Unverify showed "verified".

Align the phone path with the email path (!$user.phoneVerification ? 'unverified' : 'verified') so the toast matches the post-update state.

Fixes #1392

Test plan

  • Open a user with both email and phone in Auth
  • Verify phone → toast says verified
  • Unverify phone → toast says unverified
  • Email verify/unverify still reports correctly
Open in Web View Automation 

After invalidate, the store holds the new phoneVerification value.
Match the email path so the toast reports verified/unverified correctly.

Fixes #1392

Co-authored-by: chiragaggarwal5k <chiragaggarwal5k@gmail.com>
@cursor
cursor Bot requested a review from ChiragAgg5k August 17, 2026 03:33
@cursor
cursor Bot marked this pull request as ready for review August 17, 2026 03:33
@appwrite

appwrite Bot commented Aug 17, 2026

Copy link
Copy Markdown

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Teams feature lets you group users with membership management and role permissions

@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Corrects the phone verification success notification so it describes the refreshed verification state consistently with the email verification path.

  • Negates the refreshed phoneVerification value when selecting “verified” or “unverified” copy.
  • Leaves the underlying verification update and user-store invalidation behavior unchanged.

Confidence Score: 5/5

The PR appears safe to merge, with the corrected phone notification matching the established email verification behavior.

The change only corrects success-message selection after the user data is invalidated and refreshed, without altering verification state, authorization, or persistence behavior.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/auth/user-[user]/updateStatus.svelte Corrects the phone verification toast’s conditional to report the post-update state; no actionable defects identified.

Reviews (1): Last reviewed commit: "fix: correct phone verification success ..." | Re-trigger Greptile

@ChiragAgg5k
ChiragAgg5k merged commit e96cf47 into main Aug 17, 2026
3 of 4 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the fix/phone-verification-notification-message branch August 17, 2026 03:42
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.

🐛 Bug Report: Inconsistency in User Verification Status between Email and Phone Number

2 participants