[CP staging] Revert #94973 to fix deploy blocker #95957 (storage access modal on sign-out)#96006
Conversation
|
@eVoloshchak Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
🚧 jasperhuangg has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
…294424715-1 🍒 Cherry pick PR #96006 to staging 🍒
|
🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.4.34-0 🚀
|
|
🤖 No help site changes required. I reviewed the changes in this PR against Why:
Since there is no existing article describing this behavior and the feature is being removed, there is nothing to add or correct on the help site — so I did not open a docs PR.
|
|
🚀 Deployed to production by https://github.com/roryabraham in version: 9.4.34-14 🚀
|
|
🚀 Deployed to production by https://github.com/roryabraham in version: 9.4.34-14 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
Reverts App #94973 ("Save unsent receipts to the device gallery on sign-out") to clear deploy blocker App #95957. On iOS the storage access modal stopped appearing when signing out with a pending receipt expense. The reviewer for the forward fix (App #95906) is unavailable, so per Vit we revert the original and let the contributor re-raise.
Why the GitHub "Revert" button (a straight
git revert) doesn't work here:getPendingReceiptRequests.native.tsandsavePendingReceiptsToGalleryTest.ts.How this PR resolves it:
git rm), since the whole feature is going away. That drops 95811's edits to those two files along with them.What changed:
savePendingReceiptsToGallerymodule and the gallery-save file-download helpers added by 94973.InitialSettingsPage.tsx, restoring the prior sign-out behavior.Notes:
RECEIPT_BEARING_COMMANDSstays exported but is now unused.Fixed Issues
$ #95957
Tests
Offline tests
Same as tests (step 2 goes offline before creating the receipt expense).
QA Steps
Same as tests.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)Avatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
iOS: Native
N/A - revert of a native-only feature, verified via the Tests steps.