Repository navigation
Rename the agent rule Edit tab to Custom - #105180
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. |
|
@carlosmiceli 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] |
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppiOS: mWeb SafariMacOS: Chrome / Safari |
|
@yuwenmemon maybe merge main for the perf failing test? |
|
✋ 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! 🧪🧪
|
|
🚀 Deployed to staging by https://github.com/yuwenmemon in version: 9.5.25-0 🚀
|
|
Help site update needed. I opened Docs updates for E/A#105180 Rename agent rule Edit tab to Custom. This PR renamed the Edit tab to Custom. Step 5 in Automate Workflows with Agent Rules still named the old tab:
The docs PR changes only Edit to Custom in that sentence. Agent rules are not beta-gated in the App code, so all users see this tab. @yuwenmemon, 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/jasperhuangg in version: 9.5.25-2 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
When an admin adds an agent rule, the tab for writing their own rule is labeled "Edit", which reads like changing an existing rule (Slack). This renames it to "Custom", next to "Suggested". Only the label text changes, in all 11 locales.
Fixed Issues
$ N/A
PROPOSAL: N/A
Tests
Offline tests
Same as tests. Only the tab label changed.
QA Steps
Same as tests.
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
Not tested. Copy change only.
Android: mWeb Chrome
Not tested. Copy change only.
iOS: Native
Not tested. Copy change only.
iOS: mWeb Safari
Not tested. Copy change only.
MacOS: Chrome / Safari