Skip to content

[Due for payment 2025-03-03] [$500] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page #41398

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.

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

Closed
2 of 6 tasks
izarutskaya opened this issue May 1, 2024 · 100 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@izarutskaya
Copy link

izarutskaya commented May 1, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 1.4.69-0
Reproducible in staging?: Y
Reproducible in production?: Y
Found when executing PR : #40402
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Go to workspace settings.
  3. Go to Workflows.
  4. Go to Connect bank account.
  5. Click Connect manually.
  6. On Personal information, enter "First" and "Last" as first name and last name respectively.
  7. Proceed to Onfido flow.
  8. Accept the terms.
  9. On "Continue on your phone" page, click on the blue back button.

Expected Result:

App returns to previous page.

Actual Result:

The page loads infinitely.
On mweb, the back button is not responsive.

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6467589_1714550806295.20240501_160104.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~013867ab061a360e2f
  • Upwork Job ID: 1792533118121684992
  • Last Price Increase: 2025-01-07
Issue OwnerCurrent Issue Owner: @RachCHopkins
@izarutskaya izarutskaya added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 1, 2024
Copy link

melvin-bot bot commented May 1, 2024

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.

@izarutskaya
Copy link
Author

We think this issue might be related to the #collect project.

@melvin-bot melvin-bot bot added the Overdue label May 3, 2024
Copy link

melvin-bot bot commented May 7, 2024

@muttmuure Eep! 4 days overdue now. Issues have feelings too...

@muttmuure
Copy link
Contributor

I think this might be a site slow problem

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 7, 2024
@muttmuure
Copy link
Contributor

Can we retest this? I'm not sure it is still happening

@melvin-bot melvin-bot bot removed the Overdue label May 14, 2024
Copy link

melvin-bot bot commented May 15, 2024

@muttmuure this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label May 17, 2024
@muttmuure muttmuure added the External Added to denote the issue can be worked on by a contributor label May 20, 2024
@melvin-bot melvin-bot bot changed the title Web- Onfido - Page loads infinitely when clicking back button on "Continue on your phone" page [$250] Web- Onfido - Page loads infinitely when clicking back button on "Continue on your phone" page May 20, 2024
Copy link

melvin-bot bot commented May 20, 2024

Job added to Upwork: https://www.upwork.com/jobs/~013867ab061a360e2f

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 20, 2024
Copy link

melvin-bot bot commented May 20, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

@muttmuure
Copy link
Contributor

I can reproduce this

@melvin-bot melvin-bot bot removed the Overdue label May 20, 2024
@muttmuure
Copy link
Contributor

image

@tienifr
Copy link
Contributor

tienifr commented May 21, 2024

Seems like the same issue outlined here but useMemoryHistory was deprecated in version 14.15.0. Their source code is packaged and minified so I couldn't take a look. I think we need to file a bug to them.

@eVoloshchak
Copy link
Contributor

Can reproduce the bug on staging

I agree with @tienifr, this is a bug in onfido lib, let's file a bug to them.
@muttmuure, do we have a process for that?

@kbecciv kbecciv changed the title [$250] Web- Onfido - Page loads infinitely when clicking back button on "Continue on your phone" page [$250] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page May 24, 2024
@melvin-bot melvin-bot bot added the Overdue label May 24, 2024
Copy link

melvin-bot bot commented May 27, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Copy link

melvin-bot bot commented May 27, 2024

@eVoloshchak, @muttmuure Huh... This is 4 days overdue. Who can take care of this?

@eVoloshchak
Copy link
Contributor

@muttmuure, what is your opinion on #41398 (comment)?

@muttmuure muttmuure added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 13, 2025
Copy link

melvin-bot bot commented Feb 13, 2025

Current assignee @muttmuure is eligible for the Bug assigner, not assigning anyone new.

@muttmuure muttmuure changed the title [Requires Onfido SDK upgrade] [$500] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page [$500] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page Feb 13, 2025
@muttmuure muttmuure removed their assignment Feb 14, 2025
@muttmuure muttmuure added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Feb 14, 2025
Copy link

melvin-bot bot commented Feb 14, 2025

Triggered auto assignment to @RachCHopkins (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.

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Feb 17, 2025
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Feb 24, 2025
@melvin-bot melvin-bot bot changed the title [$500] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page [Due for payment 2025-03-03] [$500] Connect Bank - Page loads infinitely when clicking back button on "Continue on your phone" page Feb 24, 2025
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Feb 24, 2025
Copy link

melvin-bot bot commented Feb 24, 2025

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Feb 24, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.4-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2025-03-03. 🎊

For reference, here are some details about the assignees on this issue:

Copy link

melvin-bot bot commented Feb 24, 2025

@eVoloshchak / @shubham1206agra @RachCHopkins @eVoloshchak / @shubham1206agra The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]

@RachCHopkins
Copy link
Contributor

@eVoloshchak, do you mind completing the checklist, please?

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Mar 3, 2025
@eVoloshchak
Copy link
Contributor

eVoloshchak commented Mar 3, 2025

This comment summarizes the issue best: #41398 (comment)

BugZero Checklist:

  • [Contributor] Classify the bug:
Bug classification

Source of bug:

  • 1a. Result of the original design (eg. a case wasn't considered)
  • 1b. Mistake during implementation
  • 1c. Backend bug
  • 1z. Other: Bug in a third-party package (onfido-sdk-ui)

Where bug was reported:

  • 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
  • 2b. Reported on staging (eg. found during regression or PR testing)
  • 2d. Reported on a PR
  • 2z. Other:

Who reported the bug:

  • 3a. Expensify user
  • 3b. Expensify employee
  • 3c. Contributor
  • 3d. QA
  • 3z. Other:
  • [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.

    Link to comment: N/A

  • [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.

    Link to discussion: regression was not critical

  • [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template
  • [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.

    Link to issue:

Regression Test Proposal

  • Is it easy to test for this bug? Yes
  • Is the bug related to an important user flow? Yes
  • Is it an impactful bug? Yes

Test:

  1. Go to Settings -> Workspaces
  2. Select any workspace
  3. Go to Workflows.
  4. Go to Connect bank account.
  5. Click Connect manually.
  6. On the Personal information page, enter first name and last name.
  7. Proceed to Onfido flow.
  8. Accept the terms.
  9. On "Continue on your phone" page, verify that there is no blue back button present.

Do we agree 👍 or 👎

@RachCHopkins
Copy link
Contributor

Payment Summary:

  • Contributor: @shubham1206agra to be paid $500 via Newdot Manual Request
  • Contributor+: @eVoloshchak to be paid $500 via Newdot Manual Request

Upwork job here

@RachCHopkins
Copy link
Contributor

No Contributors to pay via Upwork, no contracts to complete, and the Upwork post has been closed.

@github-project-automation github-project-automation bot moved this from MEDIUM to Done in [#whatsnext] #quality Mar 3, 2025
@garrettmknight
Copy link
Contributor

$500 approved for @shubham1206agra

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2
Projects
Status: Done
Status: Tracking issues
Development

No branches or pull requests

10 participants