Repository navigation
Remove Insights page and comparison beta gates - #104595
Conversation
|
@aimane-chnaif 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] |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppios.mp4iOS: mWeb SafariMacOS: Chrome / Safariweb.mov |
|
Updated |
JmillsExpensify
left a comment
There was a problem hiding this comment.
All good on my end.
JS00001
left a comment
There was a problem hiding this comment.
lgtm, whats with all of the tab icon changes in this pr though?
|
nice, lets ship it then |
|
@luacmartins going to let you merge just in case its not meant to merge yet |
|
🚧 luacmartins 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! 🧪🧪
|
|
@luacmartins @flaviadefaria Can I check this off the list?
|
|
@izarutskaya yes please |
|
🚀 Deployed to staging by https://github.com/luacmartins in version: 9.5.26-0 🚀
|
|
@MelvinBot assign me in the linked PRs |
|
I assigned
|
|
I think automation failed here. This PR should be in prod already, see this checklist #105415 |
Explanation of Change
Remove the
insightsPageandinsightsComparegates and their disabled paths. Keep the current beta-on behavior for navigation, Home insights and comparisons on every platform.Keep this PR in draft until both betas are enabled for everyone and the native fixes have fully rolled out. Backend beta definitions remain unchanged for older clients.
Also remove the temporary desktop-only permission override introduced in #104591, which is now on
main.Fixed Issues
$ #100658
$ #100665
Tests
N/A
Offline tests
N/A
QA Steps
N/A
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)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
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari