Repository navigation
Update the policy taxes table to the new style - #94208
Conversation
Extract taxes table row and table wrapper using the shared Table component pattern.
Replace SelectionListWithModal with the shared Table component while preserving bulk actions, search, and row interactions.
|
@Krishna2323 @situchan One of you needs to 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] |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 683227a349
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
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.
|
|
This is not ready for review yet |
Skip mobile row-tap selection when item.disabled is true, matching checkbox behavior and preserving the default tax rate guard.
|
Ready for review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2e4629165c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Match the Categories page pattern by excluding optimistically deleted rates while online so the last visible row keeps its bottom border radius.
Show the disabled cursor instead of the pointer when every row is locked or otherwise unselectable.
Prevent mousedown from moving focus off the search input and restore it after table data updates, matching the SelectionList focus behavior.
Screen.Recording.2026-06-23.at.12.46.13.PM.mov
Screen.Recording.2026-06-23.at.12.45.47.PM.mov
Screen.Recording.2026-06-23.at.12.47.32.PM.mov |
|
@Krishna2323 fixed |
joekaufmanexpensify
left a comment
There was a problem hiding this comment.
Good for product
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppandroid_hybrid.mp4Android: mWeb Chromeandorid_mWeb.mp4iOS: HybridAppios_hybrid.mp4iOS: mWeb Safariios_mWeb.mp4MacOS: Chrome / Safariweb_chrome.mp4 |
|
🚧 rlinoz 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! 🧪🧪
|
|
🚀 Deployed to staging by https://github.com/rlinoz in version: 9.4.19-1 🚀
Bundle Size Analysis (Sentry): |
|
🤖 I reviewed the changes in this PR against the help site files under No help site changes are required. This PR migrates the workspace Taxes page to the shared The relevant help articles (Learn about Tracking Taxes, Track taxes) describe how tax tracking works and how to configure tax rates conceptually — none of that documented behavior changes here, so the docs remain accurate. @luacmartins, no help site PR was created since no documentation updates are needed. Please confirm this assessment reflects the current behavior — if you'd like docs changes anyway, let me know and I'll open a draft PR. |
|
🚀 Deployed to production by https://github.com/blimpich in version: 9.4.19-5 🚀
|

Explanation of Change
Migrate the policy taxes page to the shared
Tablecomponent.Fixed Issues
$ #86208
Tests
Screen.Recording.2026-06-22.at.3.28.54.PM.mov
Offline tests
N/A
QA Steps
Same as tests