[No QA] Restructure Workspace Merchant Rules examples into their own section#93490
[No QA] Restructure Workspace Merchant Rules examples into their own section#93490MelvinBot wants to merge 1 commit into
Conversation
…with sub-sections Co-authored-by: Brian Lee <brianlee-expensify@users.noreply.github.com>
|
A preview of your ExpensifyHelp changes have been deployed to https://26dfe3cc.helpdot.pages.dev ⚡️ Updated articles: |
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
@twisterdotcom 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] |
HelpDot Documentation ReviewOverall AssessmentThis PR modifies heading levels in the "Examples of Workspace Merchant Rules" section of the Workspace Merchant Rules article. It promotes a Scores Summary
Key FindingsGovernance violations introduced by the diff:
Positive aspects:
Recommendations
Files Reviewed
Review based on governance files: HELP_AUTHORING_GUIDELINES.md, HELPSITE_NAMING_CONVENTIONS.md, and TEMPLATE.md. |
| --- | ||
|
|
||
| ## Examples of Workspace Merchant Rules | ||
| # Examples of Workspace Merchant Rules |
There was a problem hiding this comment.
Heading hierarchy violation: Articles must have exactly one # heading (the article title), plus an optional # FAQ. No other # headings are allowed (HELP_AUTHORING_GUIDELINES.md, Sections 4 and 9).
This creates a second # heading in the article. Additionally, "Examples of Workspace Merchant Rules" is a noun-only/topic-only heading, which is forbidden — all headings must be task-based, starting with an action verb or question word (Section 2).
Suggested fix: Demote this to ## and rephrase as a task-based heading, e.g.:
| # Examples of Workspace Merchant Rules | |
| ## How to use Workspace Merchant Rules |
Explanation of Change
Restructures the Examples of Workspace Merchant Rules content in the Workspace Merchant Rules help article so the individual examples are easier to parse.
##sub-section to its own top-level#section.##sub-section.This gives each example a proper heading in the article hierarchy and table of contents, making the section easier to scan and navigate. No content wording was changed — only the heading levels.
Requested by
brianlee-expensifyin Slack.Fixed Issues
$
PROPOSAL:
Tests
// TODO: The human co-author must fill out the tests you ran before marking this PR as "ready for review"
// Please describe what tests you performed that validates your changed worked.
Offline tests
N/A — documentation-only change.
QA Steps
// TODO: The human co-author must fill out the QA tests you ran before marking this PR as "ready for review".
// Please describe what QA needs to do to validate your changes and what areas do they need to test for regressions.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionScreenshots/Videos
N/A — documentation-only change (heading-level restructuring of an existing help article).