-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[CP Staging] Revert "[A/B Testing][ESLint Clean Up] Use useOnyx cases for files in combinedTrackSubmit" #51245
[CP Staging] Revert "[A/B Testing][ESLint Clean Up] Use useOnyx cases for files in combinedTrackSubmit" #51245
Conversation
… combinedTrackSubmit"
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪 |
We can ignore this failing check. This occurs because we are removing the code which just migrated to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Straight revert
@fabioh8010 Can you look into this? |
@mountiny looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
Straight revert so we do not have to wait for the tests to pass |
…d-expense-flow-48787-withOnyx-migration Revert "[A/B Testing][ESLint Clean Up] Use useOnyx cases for files in combinedTrackSubmit" (cherry picked from commit 635cd11) (CP triggered by Julesssss)
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 9.0.52-2 🚀
@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes. |
@shubham1206agra Yep, will look into this 👍 Will track progress here. |
🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.0.52-5 🚀
|
Reverts #49592
This seems to fix this deploy blocker issue where expensifail accounts don't see the track expense FAB option.
This branch
Staging