-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Incorrect page is showing even there is error in bank connect flow #24371
Comments
Triggered auto assignment to @jliexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
ProposalPlease re-state the problem that we are trying to solve in this issue.An incorrect step is shown in ReimbursementAccountPage when continue is pressed and there was an error in a different step What is the root cause of that problem?To show the current step the method App/src/pages/ReimbursementAccount/ReimbursementAccountPage.js Lines 258 to 269 in b8119e9
What changes do you think we should make in order to solve the problem?When there is an issue in the validation or in the submit of the form in reimbursementAccount is stored in the onyx key REIMBURSEMENT_ACCOUNT so we could call a service that refreshes the current step in backend or merge manually updating currentStep with What alternative solutions did you explore? (Optional) |
So I reported this long time ago, and it was decided to close: |
@hungvu193 i think this is valid because the issue you mentioned does have validation in front end itself but the validation error is coming here if from backend. |
Oh, I see what you mean. |
Hi @b4s36t4 - I'm not an engineer but can you explain to me the difference between what @hungvu193 has shared and this issue? Specifically the front-end vs back-end - if the other issues have been closed, why would this one (which seems to be a similar issue) be considered a bug? Thanks! |
@jliexpensify basically in the steps of the ticket mentioned what is discussed is about the information that is in the form but they say that since it hasn't been validated there is nothing to do. The difference with this ticket is that here it is validated, it does fail when you try to save and continue(Step 4), hence, the server should be able to store the "failed" step. That is the difference between both tickets. So probably this might be half internal because achData is stored in the server but since we know in the front-end something failed in the form the server should update that as well. cc @hungvu193 Let me know if I missed something else |
Job added to Upwork: https://www.upwork.com/jobs/~01cf0cacf8a882b35c |
Current assignee @jliexpensify is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @allroundexperts ( |
Ok, awesome - thanks @daordonez11! @allroundexperts could you review this comment and share your thoughts on whether this needs to be brought internally? It sounds like it could go either way. Thanks! |
Bump @allroundexperts ! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@allroundexperts @jliexpensify this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
@allroundexperts, @jliexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Bumping @allroundexperts for an update! |
Not overdue - @grgia should this be a Monthly? |
Yes, making monthly for now |
Not overdue, removed the |
Not overdue |
not prioritizing currently |
Not overdue! |
Hi @grgia - this came up in my list of issues to assign to a Project/VIP: I can't work out where it fits in. Hoping you can help? |
Heya @grgia - just bumping this one! p.s. I'm OOO from 21st to 31st March |
Alrighty, chatted to @grgia via DM and since this is internal and very low priority, we're going to close it for focus on roadmap issues. Thanks! |
@jliexpensify Wouldn't I qualify for the reporting bonus? |
Hi @b4s36t4 thanks for the bump, I'll re-open this and get you paid for $250. |
Done, thanks! |
Job removed. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Continue with the step
.Continue Over step
and observe it takes to step 3 instead of step 2Expected Result:
Flow should be take us to the correct step if any failed validations there when updating.
Actual Result:
Taking to the wrong page even having wrong details.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.53-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Kapture.2023-08-02.at.02.31.20.1.mp4
2023-08-07.19.24.46.mp4
Expensify/Expensify Issue URL:
Issue reported by: @b4s36t4
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690924062828829
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: