Skip to content

fix: Remove GBR from the account settings on initial signup. #49004

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.

Sign up for GitHub

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

Merged
techievivek merged 11 commits into from
Oct 30, 2024
Merged

fix: Remove GBR from the account settings on initial signup. #49004

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4a5d40b
fix: Remove GBR from the account settings on initial signup.
Krishna2323 Sep 11, 2024
4d3f3c8
minor fix.
Krishna2323 Sep 11, 2024
56d75ca
filter out logins without partnerUserID.
Krishna2323 Sep 11, 2024
a13cbae
Merge branch 'Expensify:main' into krishna2323/issue/47863
Krishna2323 Sep 15, 2024
bc8c598
minor fix.
Krishna2323 Sep 15, 2024
749053a
Merge branch 'main' into krishna2323/issue/47863
Krishna2323 Oct 22, 2024
dcf9a3e
minor fixes.
Krishna2323 Oct 22, 2024
2015a28
remove .partnerUserID check.
Krishna2323 Oct 22, 2024
32aac08
minor updates.
Krishna2323 Oct 22, 2024
eb9c7db
Merge branch 'Expensify:main' into krishna2323/issue/47863
Krishna2323 Oct 25, 2024
42daf68
fix: show GBR for default login.
Krishna2323 Oct 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/libs/UserUtils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ function hasLoginListError(loginList: OnyxEntry<LoginList>): boolean {
* has an unvalidated contact method.
*/
function hasLoginListInfo(loginList: OnyxEntry<LoginList>): boolean {
return !Object.values(loginList ?? {}).every((field) => field.validatedDate);
return Object.values(loginList ?? {}).some((login) => session?.email !== login.partnerUserID && !login.validatedDate);
}

/**
Expand Down
6 changes: 4 additions & 2 deletions src/pages/settings/Profile/Contacts/ContactMethodsPage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,9 @@ function ContactMethodsPage({route}: ContactMethodsPageProps) {
// Sort the login names by placing the one corresponding to the default contact method as the first item before displaying the contact methods.
// The default contact method is determined by checking against the session email (the current login).
const sortedLoginNames = loginNames.sort((loginName) => (loginList?.[loginName].partnerUserID === session?.email ? -1 : 1));

const loginMenuItems = sortedLoginNames.map((loginName) => {
const login = loginList?.[loginName];
const isDefaultContactMethod = session?.email === login?.partnerUserID;
const pendingAction = login?.pendingFields?.deletedLogin ?? login?.pendingFields?.addedLogin ?? undefined;
if (!login?.partnerUserID && !pendingAction) {
return null;
Expand All @@ -60,7 +60,9 @@ function ContactMethodsPage({route}: ContactMethodsPageProps) {
let indicator;
if (Object.values(login?.errorFields ?? {}).some((errorField) => !isEmptyObject(errorField))) {
indicator = CONST.BRICK_ROAD_INDICATOR_STATUS.ERROR;
} else if (!login?.validatedDate) {
} else if (!login?.validatedDate && !isDefaultContactMethod) {
indicator = CONST.BRICK_ROAD_INDICATOR_STATUS.INFO;
} else if (!login?.validatedDate && isDefaultContactMethod && loginNames.length > 1) {
indicator = CONST.BRICK_ROAD_INDICATOR_STATUS.INFO;
}

Expand Down