Skip to content

[CP Staging] Revert "feat: 79048 restore reverted usd flow"#90199

Merged
arosiclair merged 1 commit into
mainfrom
revert-88193-refactor/79048-restore-reverted-usd-flow
May 11, 2026
Merged

[CP Staging] Revert "feat: 79048 restore reverted usd flow"#90199
arosiclair merged 1 commit into
mainfrom
revert-88193-refactor/79048-restore-reverted-usd-flow

Conversation

@lakchote
Copy link
Copy Markdown
Contributor

@lakchote lakchote commented May 11, 2026

Reverts #88193

$ #90144
$ #90098
$ #90109

@lakchote lakchote self-assigned this May 11, 2026
@lakchote lakchote requested review from a team as code owners May 11, 2026 13:27
@melvin-bot melvin-bot Bot requested review from JmillsExpensify and aldo-expensify and removed request for a team May 11, 2026 13:27
@melvin-bot
Copy link
Copy Markdown

melvin-bot Bot commented May 11, 2026

@aldo-expensify Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot melvin-bot Bot removed the request for review from a team May 11, 2026 13:27
Copy link
Copy Markdown
Contributor

@arosiclair arosiclair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need to revert yet

@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2026

Codecov Report

❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.

Files with missing lines Coverage Δ
src/CONST/index.ts 92.30% <ø> (ø)
src/SCREENS.ts 100.00% <ø> (ø)
...gation/AppNavigator/ModalStackNavigators/index.tsx 6.81% <ø> (-0.34%) ⬇️
src/libs/Navigation/linkingConfig/config.ts 76.92% <ø> (ø)
...mbursementAccount/ConnectedVerifiedBankAccount.tsx 0.00% <ø> (ø)
...tAccount/ReimbursementAccountVerifyAccountPage.tsx 0.00% <ø> (ø)
.../BeneficialOwnerDetailsFormSubSteps/AddressUBO.tsx 0.00% <ø> (ø)
...eficialOwnerDetailsFormSubSteps/DateOfBirthUBO.tsx 0.00% <ø> (ø)
...eneficialOwnerDetailsFormSubSteps/LegalNameUBO.tsx 0.00% <ø> (ø)
...nerDetailsFormSubSteps/SocialSecurityNumberUBO.tsx 0.00% <ø> (ø)
... and 41 more
... and 13 files with indirect coverage changes

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2a6276e997

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

prevScreen();
} else {
Navigation.goBack();
setCurrentUBOSubStep((currentSubstep) => currentSubstep - 1);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Route UBO form back action to the correct prior prompt

When a user is adding another beneficial owner (i.e., they entered the details form from ARE_THERE_MORE_UBOS) and taps back on the first details substep, this fallback always decrements to IS_ANYONE_ELSE_UBO. The previous behavior routed that case back to ARE_THERE_MORE_UBOS, so this regression now sends users to an earlier branch question and can unintentionally reset the UBO decision flow before they return to the owners list.

Useful? React with 👍 / 👎.

@arosiclair arosiclair merged commit d3113fc into main May 11, 2026
48 of 55 checks passed
@arosiclair arosiclair deleted the revert-88193-refactor/79048-restore-reverted-usd-flow branch May 11, 2026 15:40
@melvin-bot melvin-bot Bot added the Emergency label May 11, 2026
@melvin-bot
Copy link
Copy Markdown

melvin-bot Bot commented May 11, 2026

@arosiclair looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@arosiclair
Copy link
Copy Markdown
Contributor

straight revert

OSBotify pushed a commit that referenced this pull request May 11, 2026
…restore-reverted-usd-flow

[CP Staging] Revert "feat: 79048 restore reverted usd flow"

(cherry picked from commit d3113fc)

(cherry-picked to staging by jasperhuangg)
@OSBotify OSBotify added the CP Staging marks PRs that have been CP'd to staging label May 11, 2026
@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.3.69-13 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

@MelvinBot
Copy link
Copy Markdown
Contributor

No help site changes are required for this PR.

This is a revert of #88193, which was an internal code refactor of the USD reimbursement bank account flow (navigation, routing, and page component structure). The original PR did not include any documentation changes, and the revert does not alter any user-facing features, labels, or workflows documented in the help site articles under App/docs/articles.

@github-actions
Copy link
Copy Markdown
Contributor

🚧 @arosiclair has triggered a test Expensify/App build. You can view the workflow run here.

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.3.69-18 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.3.70-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 failure ❌
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

@MelvinBot
Copy link
Copy Markdown
Contributor

No help site changes are required for this PR.

This PR reverts #88193, which was an internal refactor of the USD reimbursement account flow's navigation architecture (removing page-level route constants, reverting step navigation patterns). The revert restores the previous internal code structure without changing any user-facing behavior, feature names, UI labels, or workflows.

The existing help site article Connect a US Business Bank Account describes the user-facing steps, which remain unchanged by this revert.

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.3.72-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

@MelvinBot
Copy link
Copy Markdown
Contributor

No help site changes required.

This PR reverts #88193, which was an internal refactoring of the USD reimbursement account flow (navigation structure, component organization, and animation fixes for mWeb Android). The revert does not change any user-facing behavior, UI labels, feature names, or workflows — the bank account setup experience remains identical from the end user's perspective.

Since there are no changes to user-facing functionality, no updates to App/docs/articles are needed.

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/lakchote in version: 9.3.72-1 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CP Staging marks PRs that have been CP'd to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants