[No QA] Docs updates for E/A#93848 Document commuter mileage exclusion in Distance rates - #98218
[No QA] Docs updates for E/A#93848 Document commuter mileage exclusion in Distance rates#98218MelvinBot wants to merge 1 commit into
Conversation
Co-authored-by: Fedi Rajhi <fedirjh@users.noreply.github.com>
|
The only failing check is Check independent approval — there's nothing to fix in the code. That check ("Verify peer review") isn't a test or lint failure. It fails because the PR hasn't yet received an approval from an independent reviewer (someone other than the author). It's a process gate, not a code issue, so pushing a code change won't clear it — it'll go green once an eligible reviewer approves. Every other check is green or skipped; Note this PR is intentionally on hold: the documented Exclude commutes feature is still gated behind the |
|
A preview of your ExpensifyHelp changes have been deployed to https://dbdab15d.helpdot.pages.dev ⚡️ Updated articles: |
Explanation of Change
Documents the commuter mileage exclusion feature added to workspace Distance rates in #93848 (App-side of Expensify/Expensify#642599).
The help site had no coverage of commuter exclusions. This updates the NewDot Set Distance Rates article (
docs/articles/new-expensify/workspaces/Set-distance-rates.md) to explain:Removed … commuter miles based on workspace distance settingssystem message, and the map/GPS-only requirement (Map or GPS distance required).The Exclude commutes setting is gated behind the
COMMUTER_EXCLUSIONSbeta (CONST.BETAS.COMMUTER_EXCLUSIONS) and is not yet visible in the live UI. Live verification confirmed the Distance rates Settings page currently shows only Unit, Default category, and Track tax — no Exclude commutes row. All labels in this doc were taken verbatim from the shippedsrc/languages/en.tsstrings. Please hold this PR until the feature is generally available, and confirm the labels once it's live.Fixed Issues
Docs follow-up for #93848.
Tests / QA
Docs-only change. Verify the rendered Set Distance Rates article reads correctly and the new headings/links render as expected.
@fedirjh, 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.