Skip to content

[No QA] Helpsite updates for bulk edit expenses #88247

Open
stephanieelliott wants to merge 6 commits intomainfrom
helpsite-bulk-edit-expenses
Open

[No QA] Helpsite updates for bulk edit expenses #88247
stephanieelliott wants to merge 6 commits intomainfrom
helpsite-bulk-edit-expenses

Conversation

@stephanieelliott
Copy link
Copy Markdown
Contributor

Fixes $ https://github.com/Expensify/Expensify/issues/620451
Master project issue: https://github.com/Expensify/Expensify/issues/541817

Description:

Bulk Edit Expenses — Users can now select 2+ expenses from the Reports page and edit them all at once via a new "Edit multiple" option in the bulk actions menu. This opens a side panel where users can update fields (merchant, date, amount, currency, category, tag, description, tax, reimbursable/billable status) across all selected expenses simultaneously. Changes apply optimistically with offline support.

@stephanieelliott stephanieelliott self-assigned this Apr 18, 2026
title: Edit Expenses
description: Learn how to edit individual expenses, and edit multiple expenses at once.
keywords: [bulk edit expenses, edit multiple expenses Expensify, update expenses in bulk, modify multiple expenses]
internalScope: Audience is all members. Covers editing expenses individually and in bulk bulk editing multiple expenses at once. Does not cover creating, submitting or reopening reports.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in internalScope: There is a duplicate phrase -- "in bulk bulk editing multiple expenses at once" should be "in bulk, and editing multiple expenses at once" or similar. The repeated "bulk" reads as a copy-paste error.

- **Current approver**: Can edit expenses on Outstanding reports pending their approval.
- **Workspace Admin**: Can edit expenses on any Draft or Outstanding report on the Workspace.

Expenses on Approved reports must be unapproved before they can be edited. Expenses on Paid and Done report cannot be edited.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammar: "Done report" should be "Done reports" (plural) to match the pattern of the rest of the sentence ("Paid and Done reports cannot be edited").

- Date
- Other fields like Category, Tag and Attendees

**Note:** Expenses on Approved reports must be unapproved before they can be edited. Expenses on Paid or Done reports cannot be edited. [Learn how to unapprove a report](Approve expenses](/articles/new-expensify/reports-and-expenses/Approve-Expenses).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Broken markdown link: The link syntax is malformed -- it reads [Learn how to unapprove a report](Approve expenses](/articles/...). There is a missing opening bracket and extra text. It should be:

Suggested change
**Note:** Expenses on Approved reports must be unapproved before they can be edited. Expenses on Paid or Done reports cannot be edited. [Learn how to unapprove a report](Approve expenses](/articles/new-expensify/reports-and-expenses/Approve-Expenses).
**Note:** Expenses on Approved reports must be unapproved before they can be edited. Expenses on Paid or Done reports cannot be edited. [Learn how to unapprove a report](/articles/new-expensify/reports-and-expenses/Approve-Expenses).

@stephanieelliott stephanieelliott requested a review from a team as a code owner April 18, 2026 02:57

No, Paid or Done reports are locked and cannot be reopened for editing.

## Why are some fields not editable in the Edit multiple panel?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: "iit" should be "it".

Suggested change
## Why are some fields not editable in the Edit multiple panel?
If a field is restricted for any selected expense (for example, amount on a card expense), it will be disabled in the **Edit multiple** panel.


If a field is restricted for any selected expense (for example, amount on a card expense), iit will be disabled in the **Edit multiple** panel.

## Why can't I see the Edit multiple option?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: "our" should be "or" -- "Unreported, Draft, or Outstanding expenses".

Suggested change
## Why can't I see the Edit multiple option?
**Edit multiple** only appears when you select 2 or more Unreported, Draft, or Outstanding expenses that have at least one editable field.

3. Open the Outstanding or Done report you want to retract.
4. Choose **More**.
5. Select **Retract**.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Three Dots Menu Rule violation (HELPSITE_NAMING_CONVENTIONS.md): "Choose More" is prohibited. The governance rules explicitly state: do not call it "More" and do not write "More menu." You must use: "Select the three dots (⋮)."

Suggested change
4. Select the three dots **(⋮)**.


1. In the navigation tabs (on the left on web, on the bottom on mobile) go to **Spend > Reports**.
3. Open the Outstanding or Done report you want to retract.
4. Choose **More**.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Step numbering error: Steps jump from 1 to 3 -- step 2 is missing. This appears to be a numbering typo. The sequence should be 1, 2, 3, 4.

## What happens after you retract a report

- The report status will change to Draft.
- A Retracted message appears in the report activity.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent list indentation: The bullet list has inconsistent leading spaces -- lines 38 and 40 have a leading space before the dash while lines 39 and 41 do not. This can cause unexpected rendering. All bullets should start at column 0 with no leading space.

Suggested change
- A Retracted message appears in the report activity.
- The report status will change to Draft.

- The report status will change to Draft.
- A Retracted message appears in the report activity.
- You can update, add, remove, or move expenses.
- You can resubmit the report when ready.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent list indentation: Same leading-space issue as line 38. Remove the extra space before the dash.

Suggested change
- You can resubmit the report when ready.
- You can update, add, remove, or move expenses.

@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Apr 18, 2026

Concierge reviewer checklist:

  • I have verified the accuracy of the article
    • The article is within a hub that makes sense, and the navigation is correct
    • All processes, screenshots, and examples align with current product behavior.
    • All links within the doc have been verified for correct destination and functionality.
  • I have verified the readability of the article
    • The article's language is clear, concise, and free of jargon.
    • The grammar, spelling, and punctuation are correct.
    • The article contains at least one image, or that an image is not necessary
  • I have verified the formatting of the article
    • The article has consistent formatting (headings, bullet points, etc.) with other HelpDot articles and that aligns with the HelpDot formatting standards.
    • The article has proper text spacing and breaks for readability.
  • I have verified the article has the appropriate tone and style
    • The article's tone is professional, friendly, and suitable for the target audience.
    • The article's tone, terminology, and voice are consistent throughout.
  • I have verified the overall quality of the article
    • The article is not missing any information, nor does it contain redundant information.
    • The article fully addresses user needs.
  • I have verified that all requested improvements have been addressed

For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member?

cc @strepanier03

@melvin-bot melvin-bot bot requested review from stitesExpensify and strepanier03 and removed request for a team April 18, 2026 02:58
@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Apr 18, 2026

@stitesExpensify @strepanier03 One of you needs to 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]

NOTE: It looks like docs/redirects.csv was modified in this PR. Please see this SO for special instructions on how to review changes to that file.

@stephanieelliott stephanieelliott requested review from heyjennahay and removed request for stitesExpensify and strepanier03 April 18, 2026 02:58
@github-actions
Copy link
Copy Markdown
Contributor

HelpDot Documentation Review

Overall Assessment

This PR splits Edit-Expense-Reports.md into two focused articles -- Edit-Expenses.md (bulk and single expense editing) and Edit-Reports.md (report retraction). The split aligns well with the governance principle of one primary workflow per article (HELP_AUTHORING_GUIDELINES Section 1). Both files include proper YAML frontmatter with internalScope, task-based headings, and cross-links between the two new articles. However, there are several typos, a broken link, a governance rule violation, and formatting inconsistencies that need to be resolved before merge.

Scores Summary

  • Readability: 6/10 - Multiple typos ("iit", "our" instead of "or"), a broken markdown link, and inconsistent list indentation hurt scannability and trust. Step numbering in Edit-Reports.md jumps from 1 to 3, which will confuse readers. The content itself is otherwise well-structured with clear steps.
  • AI Readiness: 8/10 - Both files have good YAML metadata with internalScope, keywords, and description. Task-based headings using "How to" and "Who can" patterns are correctly applied. One deduction: internalScope in Edit-Expenses.md contains duplicate phrasing ("in bulk bulk editing"), and the heading "What expense fields can be edited" in Edit-Expenses.md lacks a question word start ("What" is present but the phrasing reads as a noun label rather than a full question -- minor).
  • Style Compliance: 5/10 - Edit-Reports.md step 4 says "Choose More" which directly violates the Three Dots Menu Rule (HELPSITE_NAMING_CONVENTIONS.md). The governance file explicitly states: "Do not call it 'More'" and requires the exact phrasing "Select the three dots (vertical three dots symbol)". Additionally, "Done report" is missing the plural ("Done reports") in Edit-Expenses.md line 18, and "Approved reports" vs "Approved report" usage is inconsistent across both files.

Key Findings

Must fix before merge:

  • Broken markdown link (Edit-Expenses.md): The link [Learn how to unapprove a report](Approve expenses](/articles/new-expensify/reports-and-expenses/Approve-Expenses) has malformed syntax -- there is an extra (Approve expenses fragment that breaks the link entirely.
  • Three dots menu rule violation (Edit-Reports.md, step 4): "Choose More" must be replaced with Select the three dots **(three vertical dots symbol)** per HELPSITE_NAMING_CONVENTIONS.md.
  • Missing step 2 (Edit-Reports.md): Numbered steps jump from 1 to 3, which will render incorrectly in Markdown (it will auto-correct to 1, 2, 3 but the source is misleading and step 2 content appears to be missing).
  • Typo "iit" (Edit-Expenses.md, FAQ section): Should be "it".
  • Typo "our" (Edit-Expenses.md, last FAQ answer): Should be "or" -- "Unreported, Draft, our Outstanding" needs to read "Unreported, Draft, or Outstanding".

Should fix:

  • Duplicate text in internalScope (Edit-Expenses.md): "in bulk bulk editing" should be cleaned up to remove the redundancy.
  • Singular/plural mismatch (Edit-Expenses.md): "Expenses on Paid and Done report cannot be edited" -- "report" should be "reports".
  • Inconsistent list indentation (Edit-Reports.md, "What happens after you retract a report"): The first and third bullets use a leading space while the second and fourth do not, which may render inconsistently.

Positive aspects:

  • Splitting one article into two separate workflow-focused articles is the right call and follows the one-workflow-per-article rule.
  • Task-based ## headings throughout both files ("How to edit a single expense", "How to retract a report", "Who can edit expenses", etc.) align with governance requirements.
  • Both articles include proper YAML frontmatter with title, description, keywords, and internalScope.
  • Cross-linking between the two articles is well-placed and uses descriptive anchor text ("learn how to edit expenses"), following the Cross-Linking Standards.
  • Navigation instructions correctly use the unified format: "In the navigation tabs (on the left on web, on the bottom on mobile)".
  • FAQ sections use question-format headings as required.

Recommendations

  1. Fix the broken markdown link in Edit-Expenses.md -- this is the highest priority as it renders incorrectly for users.
  2. Replace "Choose More" with the three dots menu phrasing per governance rules -- this is a mandatory naming convention.
  3. Fix the step numbering gap in Edit-Reports.md (1, 3, 4, 5 should be 1, 2, 3, 4) and confirm whether a step was accidentally deleted.
  4. Correct all typos: "iit" to "it", "our" to "or", "report" to "reports".
  5. Clean up the internalScope duplicate text in Edit-Expenses.md.
  6. Normalize list indentation in Edit-Reports.md so all bullets align consistently.

Files Reviewed

  • docs/articles/new-expensify/reports-and-expenses/Edit-Expenses.md -- New file. Good structure but has a broken link, typos, and a metadata duplicate that need fixing.
  • docs/articles/new-expensify/reports-and-expenses/Edit-Reports.md -- New file. Good structure but has a three dots menu rule violation, missing step, and inconsistent indentation.
  • docs/articles/new-expensify/reports-and-expenses/Edit-Expense-Reports.md -- Deleted (replaced by the two new files above).

Note: Detailed line-by-line feedback has been provided as inline comments.

@github-actions github-actions bot changed the title Helpsite updates for bulk edit expenses [No QA] Helpsite updates for bulk edit expenses Apr 18, 2026
@OSBotify
Copy link
Copy Markdown
Contributor

A preview of your ExpensifyHelp changes have been deployed to https://c5d63010.helpdot.pages.dev ⚡️

Updated articles:

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 53a50d3ee8

ℹ️ 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".

Comment on lines +18 to +20
- **All members**: Can retract their own Outstanding and Done reports.
- **Current approver**: Can retract Outstanding reports pending their approval.
- **Workspace Admin**: Can retract any Outstanding or Done report on the Workspace.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Correct retract permissions for non-admin users

This section overstates who can retract reports: the app only allows Retract when the current user is the submitter and the report is in processing (isRetractAction), while closed/Done reports are gated by admin-only isReopenAction (src/libs/ReportSecondaryActionUtils.ts:631-670). Documenting that all members can retract Done reports, and that approvers can retract Outstanding reports, will lead users to a workflow they cannot perform.

Useful? React with 👍 / 👎.

- Date
- Other fields like Category, Tag and Attendees

**Note:** Expenses on Approved reports must be unapproved before they can be edited. Expenses on Paid or Done reports cannot be edited. [Learn how to unapprove a report](Approve expenses](/articles/new-expensify/reports-and-expenses/Approve-Expenses).
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Fix malformed markdown link to unapprove docs

The unapprove reference is malformed ([Learn how to unapprove a report](Approve expenses](/articles/...) so the link will not render correctly and users cannot navigate to the required prerequisite flow from this page.

Useful? React with 👍 / 👎.

Comment on lines +1 to +5
---
title: Edit Reports
description: Retract a submitted expense report to make changes, update expenses, and resubmit.
keywords: [New Expensify, edit expense report, retract report, update submitted report, resubmit expense report, exported report changes]
internalScope: Audience is Members and Workspace Admins. Covers retracting and updating submitted expense reports. Does not cover creating reports or detailed expense editing workflows.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Add redirect for renamed report-editing article

This change replaces Edit-Expense-Reports with Edit-Reports but does not define a redirect, while other docs still link to the old URL (for example docs/articles/new-expensify/reports-and-expenses/How-to-Delete-Expenses.md:41). Without redirect_from on the new page, existing internal/external links to the old slug will break.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants