Skip to content

Commit d67c276

Browse files
committed
fix: include phone auth in ProfileView's needsNameClaim identity check
The needsNameClaim computation (added in 8cb76da) checked authTypes for email/google/apple but missed "phone" -- a distinct auth_type value from "email" server-side, and a reachable sign-up/login path in this app (LoginInitialView/CreateNetworkView both offer "Email or phone number"). A phone-authenticated account was incorrectly treated as having no identity method bound, routing it to the claim-name flow instead of the normal change-name flow. Found in re-review before this landed anywhere.
1 parent 0ac09e1 commit d67c276

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

app/network/Main/Account/AccountNavStackView.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ struct AccountNavStackView: View {
109109
needsNameClaim: {
110110
guard let networkUser = networkUserViewModel.networkUser else { return false }
111111
let hasIdentityMethod = authTypesContains(networkUser.authTypes, "email")
112+
|| authTypesContains(networkUser.authTypes, "phone")
112113
|| authTypesContains(networkUser.authTypes, "google")
113114
|| authTypesContains(networkUser.authTypes, "apple")
114115
return !hasIdentityMethod

0 commit comments

Comments
 (0)