Docs updates for E/A#97853 Add Canadian Multiple Tax Export template - #98478
Docs updates for E/A#97853 Add Canadian Multiple Tax Export template#98478MelvinBot wants to merge 1 commit into
Conversation
Co-authored-by: Nguyen Van Duc <dukenv0307@users.noreply.github.com>
|
There's nothing to fix here — no code check is failing. The only red check is Check independent approval ( Every actual CI check ( |
|
@MelvinBot please open the PR |
|
🤖 Done — this PR is now marked Ready for review. |
|
|
||
| - **Basic export** - Essential fields including date, amount, merchant, category, and receipt URL. | ||
| - **All Data - expense level** - One row per expense with all available data fields. | ||
| - **Canadian Multiple Tax Export** - One row per expense with a separate column for each tax (such as GST and PST). This template only appears when the workspace's output currency is set to CAD. |
There was a problem hiding this comment.
UI terminology violation: "output currency" is internal wording (the outputCurrency field), not a UI label. The setting is shown to users as Workspace currency (see en.ts). Per HELPSITE_NAMING_CONVENTIONS.md, use the exact UI text and avoid internal language — reword to "the workspace's currency is set to CAD."
|
|
||
| ## Why don't I see the Canadian Multiple Tax Export template? | ||
|
|
||
| The **Canadian Multiple Tax Export** template only appears when the workspace's output currency is set to CAD. If you export expenses from a workspace that uses any other output currency, this template is hidden. When your selection includes expenses from more than one workspace, the template only appears if every selected expense belongs to a workspace that outputs in CAD. |
There was a problem hiding this comment.
UI terminology violation: "output currency" (used twice here) is internal wording, not the UI label. Users see this setting as Workspace currency. Per HELPSITE_NAMING_CONVENTIONS.md (use exact UI text, avoid internal language), replace "output currency" with "workspace currency."
| - **Basic export** - Essential fields including date, amount, merchant, category, and receipt URL. | ||
| - **All Data - expense level** - One row per expense on the report with all available data fields. | ||
| - **All Data - report level** - One row per report with all available data fields. | ||
| - **Canadian Multiple Tax Export** - One row per expense with a separate column for each tax (such as GST and PST). This template only appears when the workspace's output currency is set to CAD. |
There was a problem hiding this comment.
UI terminology violation: "output currency" is internal wording (the outputCurrency field), not a UI label. The setting is shown to users as Workspace currency (see en.ts). Per HELPSITE_NAMING_CONVENTIONS.md, use the exact UI text and avoid internal language — reword to "the workspace's currency is set to CAD."
|
|
||
| ## Why don't I see the Canadian Multiple Tax Export template? | ||
|
|
||
| The **Canadian Multiple Tax Export** template only appears when the workspace's output currency is set to CAD. If you export reports from a workspace that uses any other output currency, this template is hidden. When you bulk export reports from more than one workspace, the template only appears if every selected report belongs to a workspace that outputs in CAD. |
There was a problem hiding this comment.
UI terminology violation: "output currency" (used twice here) is internal wording, not the UI label. Users see this setting as Workspace currency. Per HELPSITE_NAMING_CONVENTIONS.md (use exact UI text, avoid internal language), replace "output currency" with "workspace currency."
HelpDot Documentation ReviewOverall AssessmentThis PR makes small, well-scoped additions to two New Expensify export articles (How to Export Expenses and How to Export Reports), documenting the new Canadian Multiple Tax Export template. Each file gets one new list item in the "What export templates can I choose from?" list and one new FAQ entry explaining the CAD output-currency condition (including multi-workspace bulk-export behavior). The changes are consistent across both files and align cleanly with the existing article structure and the governance standards.Scores Summary
Key Findings
Recommendations
Files Reviewed
|
|
The only failing check is Check independent approval, and it's not something I can fix by pushing code. This check stays red until the PR gets an independent human approval — it's not evaluating the diff. Every other check (build, What to do: this PR just needs a reviewer to approve it. Once you've reviewed the doc changes and confirmed they reflect current behavior, an independent approval will turn this check green — there's no code fix required. |
Docs updates for E/A#97853
PR #97853 adds a new default export template, Canadian Multiple Tax Export, to New Expensify. It appears in the export template menu (for both expenses and reports) when the workspace's output currency is set to CAD, producing a CSV with a separate column per tax (such as GST and PST). For bulk exports spanning multiple workspaces, it only appears when every selected item belongs to a workspace that outputs in CAD.
What changed in the help site
Updated the two New Expensify export articles that list the available export templates:
Notes
export.multipleTaxExportinen.tsandCONST.REPORT.EXPORT_OPTION_LABELS.MULTIPLE_TAX_EXPORT), which is the source of truth for the rendered label.@dukenv0307, 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.