Skip to content
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

[HybridApp] Switch to Expensify Classic values are not cleared out #43142

Closed
AndrewGable opened this issue Jun 5, 2024 · 12 comments
Closed

[HybridApp] Switch to Expensify Classic values are not cleared out #43142

AndrewGable opened this issue Jun 5, 2024 · 12 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Reviewing Has a PR in review Weekly KSv2

Comments

@AndrewGable
Copy link
Contributor

AndrewGable commented Jun 5, 2024

Problem

  1. Clicking Switch to Expensify Classic twice does not have the values cleared
  2. When the values are not cleared when opening the survey again

Solution

Always clear the "Please tell us why you are leaving" data when switching from NewDot to OldApp

Issue OwnerCurrent Issue Owner: @staszekscp
@AndrewGable AndrewGable added the Daily KSv2 label Jun 5, 2024
@AndrewGable AndrewGable self-assigned this Jun 5, 2024
@AndrewGable
Copy link
Contributor Author

cc @staszekscp

@AndrewGable
Copy link
Contributor Author

Posted a discussion for my proposed solution: https://expensify.slack.com/archives/C03TQ48KC/p1717622781938369

tl;dr we should move to API.makeRequestWithSideEffects instead of write to let the request finish before closing react native.

@staszekscp
Copy link
Contributor

Hey! I'll post a PR for this today :)

@AndrewGable
Copy link
Contributor Author

BTW we got agreement that we can use makeRequestWithSideEffects here if needed 👍

@staszekscp
Copy link
Contributor

Thanks! The code is ready, I just need to record videos, and I’m going to finish that tomorrow 😅

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Jun 7, 2024
@AndrewGable
Copy link
Contributor Author

Merged the PR, making a TestFlight now

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Jun 11, 2024
@melvin-bot melvin-bot bot changed the title [HybridApp] Switch to Expensify Classic values are not cleared out [HOLD for payment 2024-06-18] [HybridApp] Switch to Expensify Classic values are not cleared out Jun 11, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jun 11, 2024
Copy link

melvin-bot bot commented Jun 11, 2024

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

Copy link

melvin-bot bot commented Jun 11, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.81-11 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 2024-06-18. 🎊

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

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Jun 13, 2024
@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-06-18] [HybridApp] Switch to Expensify Classic values are not cleared out [HOLD for payment 2024-06-20] [HOLD for payment 2024-06-18] [HybridApp] Switch to Expensify Classic values are not cleared out Jun 13, 2024
Copy link

melvin-bot bot commented Jun 13, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.82-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 2024-06-20. 🎊

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

@AndrewGable
Copy link
Contributor Author

@cead22 is having this issue again, I think I saw it happen again once too. Mind double checking it @staszekscp?

IMG_7533

RPReplay_Final1724247548.MP4

@AndrewGable AndrewGable reopened this Aug 21, 2024
@AndrewGable AndrewGable changed the title [HOLD for payment 2024-06-20] [HOLD for payment 2024-06-18] [HybridApp] Switch to Expensify Classic values are not cleared out [HybridApp] Switch to Expensify Classic values are not cleared out Aug 21, 2024
@staszekscp
Copy link
Contributor

Sure! Do we have any logs that would indicate if the request failed or hanged?

@AndrewGable
Copy link
Contributor Author

I believe that it's the same front end issue that happened in the past where the request isn't even started because the values are there before the user entered them.

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Weekly KSv2 labels Aug 28, 2024
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 Reviewing Has a PR in review Weekly KSv2
Projects
No open projects
Archived in project
Development

No branches or pull requests

2 participants