[HOLD] mWeb- Connect Bank Account - Pressing back button at android in the middle of a step takes you back to the workspace page instead of going back to the previous step @Puneet-here

[kbecciv]

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:

1. Go to staging.new.expensify.com and log in
2. Go to bank account > add info > company step
3. Press back button of mobile

Expected Result:

ou should be navigated back to the previous step

Actual Result:

You will be navigated to the workspace page

Workaround:

Unknow

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.2.9.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/193102567-8d812de1-7932-4f37-9cfb-d59b118b642c.mov

Expensify/Expensify Issue URL:

Issue reported by: @Puneet-here

Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1663792588717789

View all open jobs on GitHub

[melvin-bot[bot]]

Triggered auto assignment to @joekaufmanexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[melvin-bot[bot]]

⚠️ Looks like this issue was linked to a possible regression on PRODUCTION here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a production regression has occurred a Root Cause Analysis is required. Please follow the instructions here.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[joekaufmanexpensify]

I don't have an android phone, but worked with @pecanoro to reproduce this in an android device.

[joekaufmanexpensify]

Discussing ideal behavior for how this should work here.

[joekaufmanexpensify]

In the meantime, going to pass to engineering to review. Also, based on this post, whatever solution we do decide on is going to be on hold pending completion of this issue.

[melvin-bot[bot]]

Triggered auto assignment to @MariaHCD (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[MariaHCD]

Demoting priority and adding to project: https://github.com/Expensify/App/projects/2

[melvin-bot[bot]]

Triggered auto assignment to @tjferriss (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[tjferriss]

I'm not able to reproduce this on web staging (v1.2.18-2) or on my iPhone app (v1.2.15-3). I am successfully taken back to the previous step. Are we sure this is still broken?

[Puneet-here]

Hi @tjferriss, it's still reproducible. You have to repro this on Android and you will need to press native back button.

[tjferriss]

@MariaHCD I'm looking for your advice here based on your previous comments. If this is still reproducible should I toss it back into the engineering pool?

[MariaHCD]

@tjferriss I see that it is reproducible if you use the native back button on an Android device. However, I don't believe we have a solution for this yet and should continue to remain on hold as part of https://github.com/Expensify/App/issues/11768. I think there's no need for another engineer to take a look at this.

[melvin-bot[bot]]

@tjferriss Whoops! This issue is 2 days overdue. Let's get this updated quick!

[tjferriss]

Ok we'll hold on for now :)

[melvin-bot[bot]]

@tjferriss Huh... This is 4 days overdue. Who can take care of this?

[melvin-bot[bot]]

@tjferriss Huh... This is 4 days overdue. Who can take care of this?

[tjferriss]

Still on hold

[JmillsExpensify]

Snagging this now that I'm on the tracking issue.

[melvin-bot[bot]]

@JmillsExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

[JmillsExpensify]

Still on hold for the tracking issue.

[melvin-bot[bot]]

@JmillsExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[JmillsExpensify]

Thanks Melvin, though we're still on hold as noted above. Adding the weekly label instead.

[JmillsExpensify]

Still on hold.

[JmillsExpensify]

Still held as we go through the design process.

[JmillsExpensify]

On hold per the larger react navigation initiative.

[melvin-bot[bot]]

Looks like something related to react-navigation may have been mentioned in this issue discussion.

As a reminder, please make sure that all proposals are not workarounds and that any and all attempt to fix the issue holistically have been made before proceeding with a solution. Proposals to change our DeprecatedCustomActions.js files should not be accepted.

Feel free to drop a note in #expensify-open-source with any questions.

[JmillsExpensify]

Still on hold for the larger navigation re-write.

[JmillsExpensify]

Still on hold for navigation.

[JmillsExpensify]

Held on navigation revamp.

[JmillsExpensify]

Switching out weekly for monthly given that the navigation project is large and has no defined ETA.