Skip to content

[RCA #64131] Expensify card - RHP back button on "Reconciliation account" does not directly go back to "Card reconciliation" #67638

Closed

Description

@jponikarchuk

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.1.88-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: https://test-management.browserstack.com/projects/2148171/test-runs/TR-318/folder/12015321/39317909/816206023?sort=%2Bidentifier
Email or phone of affected tester (no customers): applausetester+310701kh@applause.expensifail.com
Issue reported by: Applause Internal Team
Device used: Mac 15.5 / Chrome
App Component: Workspace Settings

Action Performed:

Precondition:

  • Workspace is set up with Expensify card feature.
  • Workspace is connected to QBO.
  1. Go to staging.new.expensify.com with applause.expensifail.com account
  2. Go to workspace settings > Accounting.
  3. Click Card reconciliation.
  4. Enable Continuous Reconciliation.
  5. Click Reconciliation account.
  6. Click Expensify card settlement account link.
  7. Click Reconciliation account link.
  8. Click RHP back button on Reconciliation account page.

Expected Result:

App will directly return to Card reconciliation page (production behavior).

Actual Result:

App goes back to Settlement account page. Another click on RHP back button goes back to Reconciliation account. Third click on RHP back button finally returns to Card reconciliation page.

Workaround:

Unknown

Platforms:

  • Android: App
  • Android: mWeb Chrome
  • iOS: App
  • iOS: mWeb Safari
  • iOS: mWeb Chrome
  • Windows: Chrome
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

1.mp4

View all open jobs on GitHub

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    on Aug 1, 2025
  2. melvin-bot commented on Aug 1, 2025

    @melvin-bot

    Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

  3. melvin-bot commented on Aug 1, 2025

    @melvin-bot

    Triggered auto assignment to @techievivek (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  4. melvin-bot commented on Aug 1, 2025

    @melvin-bot

    💬 A slack conversation has been started in #expensify-open-source

  5. github-actions commented on Aug 1, 2025

    @github-actions
    Contributor

    👋 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:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  6. changed the title [-]RHP back button on "Reconciliation account" does not directly go back to "Card reconciliation"[/-] [+]Expensify card - RHP back button on "Reconciliation account" does not directly go back to "Card reconciliation"[/+] on Aug 1, 2025
  7. grgia commented on Aug 1, 2025

    @grgia
    Contributor

    prob #64131

  8. changed the title [-]Expensify card - RHP back button on "Reconciliation account" does not directly go back to "Card reconciliation"[/-] [+][RCA #64131] Expensify card - RHP back button on "Reconciliation account" does not directly go back to "Card reconciliation"[/+] on Aug 1, 2025
  9. 7 remaining items

  10. ZhenjaHorbach commented on Aug 4, 2025

    @ZhenjaHorbach
    Contributor

    We introduced the backTo param on the Reconciliation Account page to address this navigation issue mentioned in the PR. However, if this is the expected behavior, we can revert the backTo implementation on that page, since this navigation issue isn't part of the original issue

    What do you think? @ZhenjaHorbach

    I'll raise the PR ASAP

    Actually yes
    Is this really a regression?
    I suppose that after pressing the back button we want to return to the previous screen
    And not jump across few screens
    So as for me the behavior in test cases is a little incorrect

    @techievivek @grgia
    What do you think ?

    But in any case, we have already opened a PR
    So if needed we will fix it immediately after confirmation

  11. techievivek commented on Aug 5, 2025

    @techievivek
    Contributor

    @grgia Assigned you as the engineer since you have the required context here.

  12. grgia commented on Aug 8, 2025

    @grgia
    Contributor

    I suppose that after pressing the back button we want to return to the previous screen
    And not jump across few screens
    So as for me the behavior in test cases is a little incorrect

    yes we shouldn't jump around screens in the described manner @ZhenjaHorbach

  13. ZhenjaHorbach commented on Aug 8, 2025

    @ZhenjaHorbach
    Contributor

    I suppose that after pressing the back button we want to return to the previous screen
    And not jump across few screens
    So as for me the behavior in test cases is a little incorrect

    yes we shouldn't jump around screens in the described manner @ZhenjaHorbach

    Thanks for your opinion !
    So In this case I think we need to update test cases and close this issue after that !
    CC: @jponikarchuk

  14. m-natarajan commented on Aug 13, 2025

    @m-natarajan

    @ZhenjaHorbach This is an exploratory issue based on this test case. There’s no step for verifying the back button, so in my view, updating the test case isn’t necessary. Correct me if I’m wrong. I’ve attached the TC steps for your reference.

    Image
  15. ZhenjaHorbach commented on Aug 14, 2025

    @ZhenjaHorbach
    Contributor

    @ZhenjaHorbach This is an exploratory issue based on this test case. There’s no step for verifying the back button, so in my view, updating the test case isn’t necessary. Correct me if I’m wrong. I’ve attached the TC steps for your reference.

    Image

    I think so !
    We can leave everything as is !
    It's just that after the latest changes the behavior has changed a bit and it has become more consistent

  16. melvin-bot commented on Aug 21, 2025

    @melvin-bot

    @grgia, @muttmuure, @ZhenjaHorbach Whoops! This issue is 2 days overdue. Let's get this updated quick!

  17. melvin-bot commented on Aug 25, 2025

    @melvin-bot

    @grgia, @muttmuure, @ZhenjaHorbach 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

  18. melvin-bot commented on Aug 27, 2025

    @melvin-bot

    @grgia, @muttmuure, @ZhenjaHorbach 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

  19. ZhenjaHorbach commented on Aug 28, 2025

    @ZhenjaHorbach
    Contributor

    @grgia @muttmuure
    Let's close this issue

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

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2EngineeringReviewingHas a PR in review

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions