[No QA] Docs updates for E/A#93852 personal card connection status and last sync - #97389
Draft
MelvinBot wants to merge 4 commits into
Draft
[No QA] Docs updates for E/A#93852 personal card connection status and last sync#97389MelvinBot wants to merge 4 commits into
MelvinBot wants to merge 4 commits into
Conversation
Co-authored-by: Maruf Sharifi <marufsharifi@users.noreply.github.com>
51 tasks
Contributor
Author
|
Nothing to fix here — the only failing check is Check independent approval (step "Verify peer review"), and every other check is passing. That check isn't a code or test failure. It fails simply because the PR hasn't received an independent human review approval yet, so there's no commit I can push to make it pass. It'll clear on its own once a reviewer approves. Since this PR is intentionally a draft (staged for the |
This comment has been minimized.
This comment has been minimized.
Contributor
|
A preview of your ExpensifyHelp changes have been deployed to https://b4af41b1.helpdot.pages.dev ⚡️ Updated articles: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Explanation of Change
Follow-up help-site updates for #93852, which adds connection health visibility (status + last sync) to bank accounts and cards in the Wallet/payment surfaces.
This PR documents the new personal card connection-status experience in the Manage Personal Cards article:
All labels were taken verbatim from the PR's
src/languages/en.tsadditions and verified against the exact UI copy.walletConnectionStatusbetaThe feature in #93852 is gated behind the
walletConnectionStatusbeta (isBetaEnabled(CONST.BETAS.WALLET_CONNECTION_STATUS)), so it is not yet visible to all users. This PR is intentionally a draft — it should be merged once the feature reaches general availability so the help site doesn't describe UI most users can't see.Related surfaces not covered here (candidate follow-ups at GA)
The same feature also adds status labels to bank accounts (Active / Incomplete / Pending / Verifying / Locked, with messages like Finish adding bank account, Please confirm test transactions, We're reviewing your documentation, This account requires attention) shown in Wallet > Bank accounts and Workspaces > [Workspace] > Workflows. These map to the existing Connect / Validate / Unlock a Business Bank Account articles and were deliberately left out of this PR to keep the scope tight and avoid documenting still-evolving beta copy across multiple files. Happy to expand into those if you'd like.
@marufsharifi, please review the files changed and confirm they reflect the current behavior. Then post a new comment saying
@MelvinBot mark this PR as ready for review.