[NoQA] Split accounting and HR connection groups#87694
[NoQA] Split accounting and HR connection groups#87694ShridharGoel wants to merge 6 commits intoExpensify:mainfrom
Conversation
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
@shubham1206agra 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] |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2f04337bef
ℹ️ 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".
src/libs/PolicyUtils.ts
Outdated
| } | ||
| if (featureName === CONST.POLICY.MORE_FEATURES.ARE_CONNECTIONS_ENABLED) { | ||
| return policy?.[featureName] ? !!policy?.[featureName] : !isEmptyObject(policy?.connections); | ||
| return policy?.[featureName] ? !!policy?.[featureName] : hasAccountingConnections(policy); |
There was a problem hiding this comment.
Preserve legacy unsupported connections in feature gating
Using hasAccountingConnections(policy) as the fallback here drops policies that only have legacy unsupported connection keys (for example generic_indirect_connection) because hasAccountingConnections only checks known accounting names plus SUPPORTED_ONLY_ON_OLDDOT. Previously, any non-empty policy.connections kept ARE_CONNECTIONS_ENABLED true. With this change, those workspaces can lose the accounting feature gate and no longer reach the accounting/legacy-redirect surfaces even though a connection still exists.
Useful? React with 👍 / 👎.
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 07d2f8979c
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d821c9aad2
ℹ️ 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".
Explanation of Change
Adds shared accounting and HR connection group helpers so accounting-only surfaces can explicitly filter out HR integrations like Gusto.
This updates exported-to filters, workspace accounting connection checks, accounting sync indicators, and accounting connection helper defaults to use accounting-only connection names.
Fixed Issues
$ #87011
PROPOSAL: N/A
Tests
git diff --check.Offline tests
N/A. This PR only updates shared constants and accounting-only filtering helpers.
QA Steps
N/A. No manual QA; foundational filtering only.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, 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)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.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. This PR has no user-facing UI changes.
Android: mWeb Chrome
N/A. This PR has no user-facing UI changes.
iOS: Native
N/A. This PR has no user-facing UI changes.
iOS: mWeb Safari
N/A. This PR has no user-facing UI changes.
MacOS: Chrome / Safari
N/A. This PR has no user-facing UI changes.