You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Onboarding - Onboarding task shows "Click Accounting" when there is no Accounting tab #104011
Version Number: 9.5.9-0 Reproducible in staging?: Yes Reproducible in production?: N/A - new feature, doesn't exist in prod If this was caught during regression testing, add the test name, ID and link from BrowserStack:#102265 Email or phone of affected tester (no customers):[email protected] Issue reported by: Applause Internal Team Bug source: Exploratory - Significant User Experience Deterioration Device used: Mac 26.5 / Chrome App Component: Other
Action Performed:
Precondition:
Enable unifiedConnections beta.
Go to staging.new.expensify.com
Log in with new Gmail account.
Click Skip on work email step.
Select Manage my team's expenses.
Complete onboarding with QBO accounting software.
Go to #admins.
Open Connect to QuickBooks Online task.
Expected Result:
Step 3 in Connect to QuickBooks Online task should be "Click Connections".
Actual Result:
Step 3 in Connect to QuickBooks Online task is "Click Accounting" when there is no Accounting tab.
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
Identify the pull request that introduced this issue and revert it.
Recommendation: Demote and fix in a follow-up. Do not revert. This matches lakchote's "Behind beta demoting" call. DeployBlockerCash is already removed.
Impact: Only the step label is wrong, and only for beta users. The "Take me to accounting" link still works because the PR redirects old Accounting URLs to the Connections page.
Confidence: High
Investigation details
Hardcoded task text:addAccountingIntegrationTask in src/languages/en.ts:3750-3765. The other 10 language files have the same text.
Caveat: The task text is generated once, when onboarding completes. Existing tasks keep the old wording. If the beta state changes later, the text can become wrong again. Wording that works with or without the beta avoids both problems.
Next Steps: Reply with @MelvinBot implement this to create a draft PR, or @MelvinBot <your feedback> to refine this analysis.
Activity
melvin-bot commentedon Oct 14, 2026
You have been assigned to this deploy blocker because you recently merged this PR: #102265
applause-bot commentedon Oct 14, 2026
While Applause doesn't embed attachments for this bug, here are the links for quick view:
https://platform.applause.com/services/links/v1/external/a970c3894adee381544c2d124e19fa64404057c8517d0ceedf9b9b075bad21bf
melvin-bot commentedon Oct 14, 2026
@lakchote @situchan you have been assigned to this deploy blocker because you reviewed the offending PR: #102265
melvin-bot commentedon Oct 14, 2026
@Krishna2323 you have been assigned to this deploy blocker because you authored the offending PR: #102265
github-actions commentedon Oct 14, 2026
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
melvin-bot commentedon Oct 14, 2026
💬 A slack conversation has been started in #expensify-open-source
lakchote commentedon Oct 14, 2026
Behind beta demoting
MelvinBot commentedon Oct 14, 2026
Deploy Blocker Investigation
Recommendation: Demote and fix in a follow-up. Do not revert. This matches
lakchote's "Behind beta demoting" call.DeployBlockerCashis already removed.Cause: Consolidate integration to one Connection screen hides the Accounting tab for
unifiedConnectionsbeta users and shows Connections instead. It did not update the onboarding task text. That text is hardcoded as "Click Accounting" in src/languages/en.ts:3759.Impact: Only the step label is wrong, and only for beta users. The "Take me to accounting" link still works because the PR redirects old Accounting URLs to the Connections page.
Confidence: High
Investigation details
addAccountingIntegrationTaskin src/languages/en.ts:3750-3765. The other 10 language files have the same text.!isUnifiedConnectionsBetaEnabled.Why not revert: The PR is a large beta-gated feature (about 100 files). The bug affects beta users only and breaks no data, payments, or navigation.
Suggested follow-up fix:
ReportUtils.tsandOnboardingFlow.ts.getAccountingConnectionsRoute(isUnifiedConnectionsBetaEnabled, policyID)from src/pages/workspace/connections/utils.ts:48-53.Caveat: The task text is generated once, when onboarding completes. Existing tasks keep the old wording. If the beta state changes later, the text can become wrong again. Wording that works with or without the beta avoids both problems.
Next Steps: Reply with
@MelvinBot implement thisto create a draft PR, or@MelvinBot <your feedback>to refine this analysis.view run
lakchote commentedon Oct 19, 2026
@Krishna2323 are you picking this up as a follow-up to #102265?
Krishna2323 commentedon Oct 20, 2026
PR is ready for review: #104402
cc: @situchan @lakchote