You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR makes a small but valid cleanup to Workspace-Merchant-Rules.md by removing a duplicated FAQ heading. The file previously contained two back-to-back headings for the same question -- one title-cased (Who Can Enable Workspace Merchant Rules?) and one sentence-cased (Who can enable Workspace Merchant Rules?) -- neither of which had answer content beneath them. Removing both empty duplicate headings is the correct fix and improves the document structure.
Scores Summary
Readability: 9/10 - The removal of two empty, duplicated headings eliminates reader confusion. A question heading with no answer is disorienting, and having two versions of it compounds the problem. This change directly improves the reading experience.
AI Readiness: 9/10 - Duplicate headings are particularly problematic for AI retrieval systems, which may index both and return empty or conflicting results. Removing them improves the document's suitability for AI-assisted search and content extraction. The heading hierarchy in the FAQ section is now clean.
Style Compliance: 9/10 - The change correctly removes the inconsistency between title case and sentence case heading styles. The remaining FAQ headings in the document use sentence case consistently, which aligns with HelpDot conventions.
Key Findings
Positive: The PR correctly identifies and removes a genuine defect -- two empty, duplicated FAQ headings that provided no value to readers.
Observation (not part of this diff): The question "Who can enable Workspace Merchant Rules?" is being fully removed rather than consolidated into a single heading with an answer. If this is an intentional decision (e.g., the information is covered elsewhere in the article or is self-evident), that is reasonable. If the question is one users commonly ask, consider adding it back as a single heading with a proper answer in a follow-up PR.
Observation (not part of this diff): The file has a few pre-existing issues unrelated to this PR, such as a duplicated step 9 in the creation instructions (lines 35 and 39) and the first ## Who Can Enable Workspace Merchant Rules? heading at line 119 that still appears in the current main branch. These are out of scope for this review but may warrant a separate cleanup.
Recommendations
No blockers -- this change is clean and ready to merge as-is.
Optional follow-up: Consider whether the "Who can enable Workspace Merchant Rules?" question should be restored as a single FAQ entry with an actual answer (e.g., "Only Workspace Admins on the Control plan can enable Workspace Merchant Rules."). This information is partially covered in the YAML internalScope metadata but is not explicitly stated in the article body.
Optional follow-up: Address the duplicated step 9 in the "How to create Workspace Merchant Rules" section, which appears to be a pre-existing issue in the file.
Files Reviewed
docs/articles/new-expensify/workspaces/Workspace-Merchant-Rules.md -- 4 lines removed (two empty duplicate FAQ headings). Change is correct and improves document quality.
github-actionsbot
changed the title
Update Workspace-Merchant-Rules.md
[No QA] Update Workspace-Merchant-Rules.md
Mar 11, 2026
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
@RachCHopkins Please 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]
@RachCHopkins looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix duplicated line in doc