[No QA] Docs updates for E/A#97333 AUS Corpay certification TMD, illion report, bank statement - #98209
Conversation
…ank statement) Co-authored-by: Maxence Coulibaly <madmax330@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
|
I checked the CI on this PR. Every code/content check is passing ( The only red check is Check independent approval, which isn't a code failure — it's a gating check that stays red until an independent reviewer approves the PR. It can't be resolved with a commit; it clears once this docs update gets a review/approval. Next step: review the changed help article and, if it looks good, approve the PR (or reply |
madmax330
left a comment
There was a problem hiding this comment.
LGTM, all you @stephanieelliott
| --- | ||
|
|
||
| # Step 3: Confirm Test Transactions | ||
| # Step 3: Review the Disclosure Paperwork and Provide a Bank Statement |
There was a problem hiding this comment.
Heading rule violation (multiple # headings + non-task-based heading): Per HELP_AUTHORING_GUIDELINES.md Sections 2 and 4, an article may have only one # heading (the title) plus an optional # FAQ — all other sections must use ##. Additionally, Step N: prefixes are a forbidden generic/step-style heading. Change this newly added heading to a task-based ##, e.g. ## How to review the disclosure paperwork and provide a bank statement.
There was a problem hiding this comment.
Done in 44de7a1 — converted to a task-based ## heading: ## How to review the disclosure paperwork and provide a bank statement. Also converted the pre-existing # Step 1/2 and renumbered # Step 4/5 headings to task-based ## so the whole article now follows the single-#-heading rule.
|
|
||
| --- | ||
|
|
||
| # Step 4: Confirm Test Transactions |
There was a problem hiding this comment.
Heading rule violation: This renumbered heading still uses a second # heading with a Step N: prefix. Per HELP_AUTHORING_GUIDELINES.md Sections 2 and 4, use a single # title and convert this to a task-based ##, e.g. ## How to confirm test transactions.
There was a problem hiding this comment.
Done in 44de7a1 — now ## How to confirm test transactions.
| --- | ||
|
|
||
| # Step 4: Start Sending Global Reimbursements | ||
| # Step 5: Start Sending Global Reimbursements |
There was a problem hiding this comment.
Heading rule violation: This renumbered heading still uses a second # heading with a Step N: prefix. Per HELP_AUTHORING_GUIDELINES.md Sections 2 and 4, use a single # title and convert this to a task-based ##, e.g. ## How to start sending global reimbursements.
There was a problem hiding this comment.
Done in 44de7a1 — now ## How to start sending global reimbursements.
|
|
||
| * ✅ Address verification document | ||
| * ✅ Bank statement for direct debit setup | ||
| * ✅ Recent bank statement (dated within the last three months) for the business bank account you're connecting |
There was a problem hiding this comment.
Redundancy / clarity: This "Recent bank statement (dated within the last three months)" requirement duplicates the existing "Bank statement for direct debit setup" line above and is also restated in the new Step 3 body (line 83). Per HELP_AUTHORING_GUIDELINES.md Section 6 (avoid vague/duplicated wording), consolidate the two bank-statement bullets into one clear requirement to avoid confusing the reader about how many statements are needed.
There was a problem hiding this comment.
Done in 44de7a1 — consolidated the two bullets into one: ✅ Recent bank statement (dated within the last three months) for the business bank account you're connecting, used for direct debit setup. It's now stated once in Document Requirements and referenced once in the setup flow.
|
🤖 There's nothing to fix here in code — all real CI checks are passing ( The only failing check is Check independent approval (the To clear it: this draft needs to be marked ready for review and approved by an independent reviewer. Since the PR description already asks you to confirm the docs, once you're satisfied you can comment |
HelpDot Documentation ReviewOverall AssessmentThis PR updates the Global Reimbursement - Australia article to reflect the AUS Corpay certification changes: a new TMD disclosure document, an illion report signer requirement, and a recent bank statement requirement. The changes are accurate, well-scoped to a single workflow, and factually clear. The main issues stem from the article pre-existing structural pattern (multiple top-level Step N headings), which the new content extends rather than corrects. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 69213d74cf
ℹ️ 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".
|
|
||
| **Note:** For global reimbursements, Expensify requires information from at least two Directors or Senior Officers for Australian companies. | ||
|
|
||
| **Note:** Both signers must be listed as directors on the company's illion report. They can't be any two people at the business. |
There was a problem hiding this comment.
Remove Senior Officer option from AUD signer docs
With this new AUD-specific note saying both signers must be directors on the illion report, the surrounding instructions still tell users they can enter a “Director or Senior Officer” and that Expensify requires two “Directors or Senior Officers.” The current AUD signer flow only asks for director emails and tells users both signers must be listed as directors on the illion report (src/languages/en.ts:4314-4319, rendered in src/pages/ReimbursementAccount/NonUSD/SignerInfo/EnterEmail.tsx:92-95), so leaving the Senior Officer wording in the same section makes the help article internally contradictory for Australian onboarding.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Good catch — done in 44de7a1. Removed the "or Senior Officer" wording from all three spots in the Add Director Information section so it now consistently says "Director(s)", matching the current AUD signer flow (which only collects director emails and requires both signers be listed as directors on the illion report).
|
🤖 Nothing has changed since my previous comment — the only failing check is still Check independent approval, which fails because the PR has no independent human approval yet. It is not caused by anything in the diff and no code push can fix it. All real CI checks ( To clear it, this draft needs to be marked ready for review and approved by an independent reviewer. |
|
@MelvinBot can you address the review feedback? |
…ent requirement, drop Senior Officer wording Co-authored-by: Maxence Coulibaly <madmax330@users.noreply.github.com>
|
Addressed all the review feedback in 44de7a1: Heading rule (github-actions bot): Converted every
(I converted Steps 1 & 2 too, not just the renumbered ones, so the whole file is consistent rather than half-migrated.) Redundancy (github-actions bot): Consolidated the two bank-statement bullets in Australia Document Requirements into one — Remove Senior Officer wording (Codex, P2): Dropped "or Senior Officer" from all three spots in Add Director Information. I verified against the current AUD signer flow ( Since this is a content-only edit to an existing article (no routing/config or new vocabulary), there were no additional required local checks to run beyond what CI covers. Let me know if you'd like any further tweaks. |
|
🤖 Still nothing to fix in code — the situation is unchanged since my previous comment. All real CI checks pass; the only failing one is Check independent approval ( To clear it: mark this draft ready for review and get an independent reviewer to approve it. |
|
A preview of your ExpensifyHelp changes have been deployed to https://019a12fd.helpdot.pages.dev ⚡️ Updated articles: |
|
🚀 Deployed to staging by https://github.com/stephanieelliott in version: 9.4.55-0 🚀
|
|
🚀 Deployed to production by https://github.com/Beamanator in version: 9.4.55-4 🚀
Bundle Size Analysis (Sentry): |
Docs updates for E/A#97333
This draft updates the NewDot Global Reimbursement - Australia help article to reflect the AUS Corpay certification changes introduced in #97333.
What changed in the PR
The PR updated the AUD (Australia) non-USD bank account / Corpay setup flow with three user-facing changes:
Help site changes made
Updated
docs/articles/new-expensify/wallet-and-payments/Global-Reimbursement-Australia.md:@madmax330, 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.