[No QA] Docs updates for E/A#88181 Update search groupings and suggested search names#88415
[No QA] Docs updates for E/A#88181 Update search groupings and suggested search names#88415trjExpensify merged 15 commits intomainfrom
Conversation
…ches Updates terminology across help site articles to match the new search section names and suggested search labels from E/A#88181: - To-do/Explore sections replaced with Expense reports/Accounting - Submit -> Drafts, Approve -> Needs approval, Pay -> Ready to pay - Export -> Awaiting export - Monthly accrual/Unapproved cash/card -> Accounting/Cash accruals/Card accruals - Reconciliation -> Bank reconciliation Co-authored-by: Aimane Chnaif <aimane-chnaif@users.noreply.github.com>
|
A preview of your ExpensifyHelp changes have been deployed to https://6a70c8b8.helpdot.pages.dev ⚡️ Updated articles: |
|
@stephanieelliott Spend is on prod #88010 |
|
Jumping on this one now, will resolve the conflicts in Cursor since they are pretty complex. |
- Approve-Expenses: keep reject-report intro; use Spend > Approve; clarify For you copy - Unapproved-Spend, Pay-Expenses, Reimbursement-Payment-Methods: use main nav paths - Using-Search-on-the-Spend-Page: take main To-do/Explore model; expand keywords Made-with: Cursor
|
@trjExpensify we need to rename the page Typically we would rename it to match the label (so "Accounting"), but I feel like that's going to be confusing - this is page describes spend, rather than information about integrations. |
HelpDot Documentation ReviewOverall AssessmentThis PR updates four help articles to reflect renamed search groupings and suggested search labels (To-do becomes Expense reports, Explore is removed, Monthly accrual becomes Accounting, and shortcuts are renamed). The changes are well-scoped and the new Reconcile-Expenses-With-Accounting.md article is a solid replacement for the deleted Unapproved-Spend.md. However, there are a few governance violations that should be resolved before merge. Scores Summary
Key FindingsMust fix:
Should fix: Positive aspects:
Recommendations
Files Reviewed
Review based on governance files: HELPSITE_NAMING_CONVENTIONS.md, HELP_AUTHORING_GUIDELINES.md, and TEMPLATE.md. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bbcc91e8f1
ℹ️ 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".
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…-Search-Shortcuts.md
| ## What each Accounting search shortcut shows you | ||
|
|
||
| - **Awaiting export**: Reports that are approved and ready to be exported to your accounting system. | ||
| - **Cash accruals**: Reimbursable expenses that are not yet approved and may need to be accrued. |
There was a problem hiding this comment.
AI readiness (heading rule): Per HELP_AUTHORING_GUIDELINES.md Section 2, all ## headings must start with an action verb or question word (How, What, Where, Who, Why, When). "What each Accounting search shortcut shows you" does start with "What," but it reads more as a descriptive label than a user question. Consider rephrasing to a clearer question format:
## What does each Accounting search shortcut show you
HelpDot Documentation ReviewOverall AssessmentThis PR replaces two older documentation articles with two well-structured new articles ("Accounting Search Shortcuts" and "Use Search Shortcuts") and updates navigation references in the existing "Approve Expenses" article to reflect UI changes (e.g., "Reports > Expenses" becomes "Spend > All expenses"). Deleted articles are properly redirected. Overall, this is a strong set of changes that aligns well with governance standards. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: This review covers only the proposed changes in the diff, not unchanged portions of files. |
|
@trjExpensify looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
|
|
||
| # Accounting Search Shortcuts | ||
|
|
||
| The **Accounting** search shortcuts on the **Spend** page bring together the expense views you need to reconcile your records and close your books. It helps you identify what’s ready to export, what still needs attention, and what should be matched against statements or bank data. |
There was a problem hiding this comment.
Readability: Subject-verb disagreement. "The Accounting search shortcuts...bring together..." (plural) is followed by "It helps you identify..." (singular). Change "It helps" to "They help" to match the plural subject.
| 2. In the **Accounting** section, choose a search shortcut based on what you need to review. | ||
| 3. Use filters like **Date**, **Feed**, or **Posted** to refine your view. | ||
| 4. Review the **Total** in the footer to understand the amount for that view. | ||
|
|
There was a problem hiding this comment.
Naming and style (navigation phrasing): Per HELPSITE_NAMING_CONVENTIONS.md, the standard unified navigation sentence is: "Click the navigation tabs (on the left on web, on the bottom on mobile)." This line uses "In the navigation tabs (on the left on web, on the bottom on mobile) select Spend" which deviates from the standard. Also, a comma is missing after the closing parenthesis before "select".
Suggested fix: "Click the navigation tabs (on the left on web, on the bottom on mobile), then select Spend."
| 1. In the navigation tabs (on the left on web, on the bottom on mobile), select **Spend**. | ||
| 2. In the **Expense reports** section, choose a search shortcut. | ||
|
|
||
| Shortcuts include: |
There was a problem hiding this comment.
Naming and style (navigation phrasing): Per HELPSITE_NAMING_CONVENTIONS.md, the standard unified navigation sentence is: "Click the navigation tabs (on the left on web, on the bottom on mobile)." This line uses "In the navigation tabs (on the left on web, on the bottom on mobile), select Spend" which deviates from the standard.
Suggested fix: "Click the navigation tabs (on the left on web, on the bottom on mobile), then select Spend."
| # Accounting Search Shortcuts | ||
|
|
||
| The **Accounting** search shortcuts on the **Spend** page bring together the expense views you need to reconcile your records and close your books. It helps you identify what’s ready to export, what still needs attention, and what should be matched against statements or bank data. | ||
|
|
There was a problem hiding this comment.
Readability: The second paragraph ("Workspace Admins can use these shortcuts to validate totals, track liabilities, and ensure your expenses align...") switches from third person "Workspace Admins" to second person "your expenses." Consider using consistent second person throughout, e.g., "You can use these shortcuts to validate totals, track liabilities, and ensure expenses align with your accounting records before and after export." (The "Who can use" section already establishes the audience as Workspace Admins.)
| --- | ||
|
|
||
| ## What each Accounting search shortcut shows you | ||
|
|
There was a problem hiding this comment.
Cross-linking (link placement): Per HELP_AUTHORING_GUIDELINES.md Section 7, "Do not place links inside numbered step instructions. Do not interrupt procedural flow with inline links." This link appears immediately after the numbered steps and is part of the procedural section. This is borderline acceptable since it's after the steps rather than within them, but consider moving it to a separate paragraph with a clearer visual break from the steps.
| - You can identify reports that are ready to export to your accounting system. | ||
| - You can account for unapproved expenses as accruals before closing a period. | ||
| - You can verify imported credit card expense totals against statements and bank data. | ||
| - You can ensure your records are complete before and after reconciliation. |
There was a problem hiding this comment.
Structure / formatting: The bullet list items in this section have a leading space before the dash ( - You can...). While this renders correctly in most Markdown parsers, standard Markdown convention is to start list items without the leading space for consistency with the rest of the article (e.g., the bullet list under "What each Accounting search shortcut shows you" on line 43 has no leading space).
|
|
||
| Search shortcuts on the **Spend** page apply pre-built filters that help you quickly find expense reports that need action. These shortcuts are automatically tailored to your role and show only the reports that require your attention. | ||
|
|
||
| To create your own custom searches with advanced filtering, [learn how to use search operators](/articles/new-expensify/reports-and-expenses/Using-search-operators). |
There was a problem hiding this comment.
Cross-linking (link placement): Per HELP_AUTHORING_GUIDELINES.md Section 7, links should be placed "After explanatory sentences" and "In conditional dependency statements," not in introductory paragraphs that precede the main content. This link to search operators in the intro could dilute the primary workflow focus. Consider moving it to the end of the article or to a "Related articles" section.
|
|
||
| All members can use search shortcuts, but what you see depends on your role: | ||
|
|
||
| - **Submitters:** See **Drafts** when reports are ready to submit |
There was a problem hiding this comment.
Structure / formatting: The bullet list items here use a leading space before the dash ( - **Submitters:**...), but the list under "How to use search shortcuts" on line 36 does not. Use consistent list formatting throughout the article -- either all with or all without the leading space.
HelpDot Documentation ReviewOverall AssessmentThis PR replaces two older articles (Unapproved-Spend.md and Using-Search-on-the-Spend-Page.md) with two well-structured new articles (Accounting-Search-Shortcuts.md and Use-Search-Shortcuts.md), updates navigation references in Approve-Expenses.md to match the current UI (e.g., Reports to Spend, Expenses to All expenses), and adds proper redirects. The new articles are a significant quality improvement over their predecessors, following governance standards closely with correct heading hierarchy, complete metadata, and clear procedural steps. Scores Summary
Key FindingsPositive aspects:
Items to consider (minor):
Observation (not scored, outside diff scope):
Recommendations
Files Reviewed
This review assessed only the proposed changes in the PR diff, per HelpDot governance files. |
|
@MelvinBot why was this PR reviewed multiple times by |
|
The "PR Reviews with Claude Code" workflow runs on every push to the PR branch, not just the initial submission. Each run posts new review comments as This PR had many pushes (15 commits across Apr 23–24), and the workflow ran at least 5 times, producing 38 total review comments in batches that correlate with each push:
This is expected behavior for any PR with multiple pushes — the workflow re-reviews on each |
|
🚀 Deployed to staging by https://github.com/trjExpensify in version: 9.3.62-5 🚀
Bundle Size Analysis (Sentry): |
|
🚀 Deployed to staging by https://github.com/trjExpensify in version: 9.3.64-0 🚀
Bundle Size Analysis (Sentry): |
Summary
Files changed
docs/articles/new-expensify/reports-and-expenses/Using-Search-on-the-Reports-Page.md— Major rewrite to match new section structuredocs/articles/new-expensify/reports-and-expenses/Unapproved-Spend.md— Updated navigation paths and view namesdocs/articles/new-expensify/reports-and-expenses/Approve-Expenses.md— Updated navigation pathdocs/articles/new-expensify/wallet-and-payments/Pay-Expenses.md— Updated navigation pathsdocs/articles/new-expensify/wallet-and-payments/Reimbursement-Payment-Methods.md— Updated navigation pathsTest plan