Repository navigation
[CP both] Show Talk to a human without the beta flag - #104152
Conversation
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. |
|
🚧 chiragsalian 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! 🧪🧪
|
…beta [CP both] Show Talk to a human without the beta flag (cherry picked from commit 7e9ad84) (cherry-picked to staging by jasperhuangg)
|
🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.5.10-1 🚀
|
|
A help site update is needed. This PR removes the I created Docs updates for E/A#104152 Add Talk to a human option to Concierge Basics. It adds one sentence to the "Can I talk to a real person instead of Concierge?" answer in Concierge Basics, which currently says:
The edit adds: "You can also select Account > Help > Talk to a human." I checked the label against @jasperhuangg, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR |
|
🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.5.10-1 🚀
|
…beta [CP both] Show Talk to a human without the beta flag (cherry picked from commit 7e9ad84) (cherry-picked to production by jasperhuangg)
|
A help site update is needed, and the docs PR already exists: [No QA] Docs updates for E/A#104152 Add Talk to a human option to Concierge Basics. I didn't create a duplicate. This PR removes the This time I also checked the live UI. The Help page shows Talk to a human with the description "Get help from a real person", between Concierge and Help site. That matches the docs edit. @jasperhuangg, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR |
|
🚀 Deployed to production by https://github.com/cristipaval in version: 9.5.11-0 🚀
Bundle Size Analysis (Sentry): |
|
🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.5.12-0 🚀
|
|
A help site update is needed, and the docs PR already exists: [No QA] Docs updates for E/A#104152 Add Talk to a human option to Concierge Basics. I didn't create a duplicate. This PR removes the I've now checked the label in the live UI. Account > Help shows Concierge, Talk to a human ("Get help from a real person"), and Help site, in that order. @jasperhuangg, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR |
|
🚀 Deployed to production by https://github.com/cristipaval in version: 9.5.12-1 🚀
Bundle Size Analysis (Sentry): |
|
🚀 Cherry-picked to staging by https://github.com/jasperhuangg in version: 9.5.13-0 🚀
|
|
A help site update is needed, and the docs PR already exists: [No QA] Docs updates for E/A#104152 Add Talk to a human option to Concierge Basics. I didn't create a duplicate. This PR removes the I confirmed the label in the live UI. The Help page lists Concierge, Talk to a human ("Get help from a real person"), and Help site. @jasperhuangg, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR |
|
🚀 Deployed to production by https://github.com/grgia in version: 9.5.13-6 🚀
Bundle Size Analysis (Sentry): |


Explanation of Change
Fixed Issues
N/A
PROPOSAL: N/A
Tests
Not run locally. Reviewer steps:
Offline tests
QA Steps
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