Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.32k stars 2.75k forks source link

[Guided Setup] Sign in – Able to dismiss Onboarding modal when click on browser back button #47246

Closed IuliiaHerets closed 1 week ago

IuliiaHerets commented 1 month ago

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.0.19-0 Reproducible in staging?: Y Reproducible in production?: N Email or phone of affected tester (no customers): ponikarchuks+1812824@gmail.com Issue reported by: Applause Internal Team

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. In Sign in page enter new Gmail account
  3. When Onboarding modal appears click on browser back button

Expected Result:

Onboarding modal is present

Actual Result:

Onboarding modal disappears

Workaround:

Unknown

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/a463e86e-280b-4fab-a537-9524d81653b8

View all open jobs on GitHub

melvin-bot[bot] commented 1 month ago

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

melvin-bot[bot] commented 1 month ago

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

github-actions[bot] commented 1 month ago

:wave: 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.
IuliiaHerets commented 1 month ago

We think that this bug might be related to #wave-collect - Release 1

github-actions[bot] commented 1 month ago

:wave: 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.
melvin-bot[bot] commented 1 month ago

Current assignee @deetergp is eligible for the DeployBlockerCash assigner, not assigning anyone new.

github-actions[bot] commented 1 month ago

:wave: 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.
mountiny commented 1 month ago

@filip-solecki Could you have a look into this in the morning please?

Beamanator commented 1 month ago

Should we call this NAB if we're ok waiting till morning for a resolution? 🤔

deetergp commented 1 month ago

It really doesn't seem like a blocker…

Beamanator commented 1 month ago

Agreed! Marking NAB!±

filip-solecki commented 1 month ago

Me and @adamgrzybowski are not able to reproduce it, did you try to clear Onyx data?

mountiny commented 1 month ago

Asked in QA if they can repro this on mac too https://expensify.slack.com/archives/C9YU7BX5M/p1723547272644439 the video is from windows

kavimuru commented 4 weeks ago

Repro in Mac Safari and Mac chrome and mweb

https://github.com/user-attachments/assets/d68b7f7a-8d55-4bf2-955f-dbcf53cdad61

https://github.com/user-attachments/assets/20241e9e-5867-41d2-9886-a71e1a8097f4

adamgrzybowski commented 4 weeks ago

https://github.com/user-attachments/assets/d2d566ba-28ce-4e46-8a16-59160f8a89e1

As you can see I can't reproduce it on my end. The difference I noticed in my and your case is that the URL with the report id shows briefly in your.

Not sure why this happens but I won't be able to debug this issue without reliable reproduction steps.

I can point you to this function https://github.com/Expensify/App/blob/c5feb89f9f02cfb303e86246408f40c4f5a23119/src/libs/Navigation/AppNavigator/createCustomStackNavigator/CustomRouter.ts#L103-L118

I would guess this value const targetFocusedRoute = findFocusedRoute(action?.payload); is wrong. Could you please check it with the debugger? Maybe I will see what is wrong

deetergp commented 4 weeks ago

@kavimuru Were your reproduction steps on Mac any different than what is listed in the GH description?

adamgrzybowski commented 3 weeks ago

This may be a race condition. It looks like the initial chat doesn't have time to load and break the behavior on my end.

melvin-bot[bot] commented 3 weeks ago

@deetergp, @anmurali Eep! 4 days overdue now. Issues have feelings too...

deetergp commented 3 weeks ago

@kavimuru Bump!

melvin-bot[bot] commented 2 weeks ago

@deetergp @anmurali 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[bot] commented 2 weeks ago

@deetergp, @anmurali Eep! 4 days overdue now. Issues have feelings too...

deetergp commented 2 weeks ago

I reached out to @kavimuru in Slack.

deetergp commented 1 week ago

I asked for a retest in Slack but have not yet heard back.

melvin-bot[bot] commented 1 week ago

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

trjExpensify commented 1 week ago

I asked for a retest in Slack but have not yet heard back.

@deetergp can you link the thread when making these kind of references, just helps follow the breadcrumbs. Where are we at with this re-test?

deetergp commented 1 week ago

Of course! This is where I asked previously https://expensify.slack.com/archives/C9YU7BX5M/p1724783266169849, but I think it got lost in the shuffle. Asked again here https://expensify.slack.com/archives/C9YU7BX5M/p1725431001409449

m-natarajan commented 1 week ago

Not reproducible

https://github.com/user-attachments/assets/55d61b2d-79c1-44aa-bedd-3f411f4983c2

trjExpensify commented 1 week ago

Cool, so then let's close this out @deetergp?