Revert #92007 Concierge AI Feature Promo Modal#93977
Conversation
Codecov Report❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.
|
|
🚧 @blimpich has triggered a test Expensify/App build. You can view the workflow run here. |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
@mjasikowski do you want QA team to retest all linked blockers? |
|
🚀 Cherry-picked to staging by https://github.com/puneetlath in version: 9.4.15-2 🚀
Bundle Size Analysis (Sentry): |
|
🤖 No help site changes required. This PR reverts #92007, removing the AI features promo modal flow (the modal itself, its navigation guard/routes, animations, and the related The promo modal was a transient in-app onboarding/promotion element — it is not documented anywhere under Because no documented behavior changes, I did not create a draft help site PR. @mjasikowski, there is no linked help site PR to review since no docs changes are needed. If you believe a help article should be updated to reflect this behavior, let me know and I'll draft one. |
|
🚀 Deployed to production by https://github.com/puneetlath in version: 9.4.15-3 🚀
|
|
🚀 Cherry-picked to staging by https://github.com/puneetlath in version: 9.4.16-1 🚀
Bundle Size Analysis (Sentry): |
|
🤖 Help site review: no docs changes required. This PR reverts #92007, which added the in-app Concierge AI Feature Promo Modal — a one-time promotional/training modal (with Lottie animations, a navigation guard, routes, and localized strings) that advertised existing AI features. I reviewed the change against the help site files under
Because the revert removes a transient promotional UI element and does not change any documented feature's availability or behavior, the existing help site content remains accurate. I did not create a draft PR. @mjasikowski, since no help site changes are required, there is no linked help site PR to review. If you believe an article should still be updated as a result of this revert, let me know and I'll create a draft PR. |
|
🚀 Deployed to production by https://github.com/blimpich in version: 9.4.16-5 🚀
|
Explanation of Change
This reverts #92007.
The revert removes the AI features promo modal flow, related navigation guard and routes, added animations, and related language/constants updates, restoring the previous behavior from
mainbefore that PR was merged.Fixed Issues
#93964
#93968
#93969
#93970
#93973
#93974
#93976
Tests
Offline tests
[No QA]
QA Steps
[No QA]
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, 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.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.