[CP Staging] Revert server-driven AZ status on expense reports#97003
Conversation
|
@sobitneupane 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] |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🚧 yuwenmemon 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! 🧪🧪
|
[CP Staging] Revert server-driven AZ status on expense reports (cherry picked from commit b3e7147) (cherry-picked to staging by yuwenmemon)
…125181995-1 🍒 Cherry pick PR #97003 to staging 🍒
|
🚀 Cherry-picked to staging by https://github.com/yuwenmemon in version: 9.4.44-6 🚀
|
|
🤖 I reviewed the changes in this PR against the help site files under No help site changes are required, so I did not create a draft PR. Why: This PR is a revert of #95959. It stops non-chat reports (expense reports) from mounting the AgentZero status gate, which was causing a Concierge "thinking" indicator to flash briefly during unrelated actions like inviting a member. This is an internal UI/implementation fix that removes unintended behavior — it does not add or change any user-facing feature, setting, tab, or workflow that the help site documents. I checked the help articles that reference Concierge (e.g. If you believe a specific article should be updated, reply with the article and the behavior it should describe and I'll draft the changes. |
@roryabraham please review
Explanation of Change
Reverts #95959, which lets non-chat reports mount the AgentZero status gate. On expense reports this makes a Concierge "thinking" indicator flash briefly during unrelated actions such as inviting a member, which is a deploy blocker.
Because a large report-list refactor landed on top of #95959, this could not be reverted automatically. The AgentZero/Concierge changes were undone by hand while preserving that refactor: the expense-report tail indicator and its provider wrappers are removed, and the
ReportActionsListTailIndicatorrename is reverted back toReportActionsListHeader.Fixed Issues
$ #96979
PROPOSAL: N/A
Tests
Offline tests
Same as tests.
QA Steps
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, 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)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
N/A — revert.
Android: mWeb Chrome
N/A — revert.
iOS: Native
N/A — revert.
iOS: mWeb Safari
N/A — revert.
MacOS: Chrome / Safari
N/A — revert.