[No QA] Docs updates for E/A#89422 Update Copilot help docs for top-level menu#90805
[No QA] Docs updates for E/A#89422 Update Copilot help docs for top-level menu#90805MelvinBot wants to merge 4 commits into
Conversation
Update Copilot-Access.md and Navigating-New-Expensify.md to reflect that Copilot is now a dedicated top-level item under Account settings instead of being nested under Security. Updates navigation paths, section references, and the Switch to Copilot Account instructions to match the new inline Switch button on the Copilot page. Co-authored-by: Wildan M <wildan-m@users.noreply.github.com>
HelpDot Documentation ReviewOverall AssessmentThis PR updates Copilot help documentation to reflect a UI change: Copilot settings have moved from Account > Security to a dedicated Account > Copilot top-level menu item. The changes are well-scoped, accurate, and include several incidental quality improvements (smart quote cleanup, three-dot menu formatting). The PR correctly updates navigation paths, button labels, and section references to match the new UI. Scores Summary
Key FindingsPositive changes:
Pre-existing issues (not introduced by this PR, noted for awareness):
Recommendations
Files Reviewed
This review assessed only the proposed changes in the PR diff. Pre-existing issues are noted for awareness but did not affect scoring. |
| --- | ||
|
|
||
| # Remove or Change a Copilot’s Access | ||
| # Remove or Change a Copilot's Access |
There was a problem hiding this comment.
Heading hierarchy violation: Per HELP_AUTHORING_GUIDELINES.md Section 4, an article may have exactly one # heading (the article title) plus an optional # FAQ. All other sections must use ##. This heading — and the other # headings in this file (# What Can a Copilot Do?, # Add a Copilot, # Switch to a Copilot Account, # Forward Receipts as a Copilot) — should be demoted to ##.
This is a pre-existing issue, but since this line is being modified it is a good opportunity to fix it across the file.
|
A preview of your ExpensifyHelp changes have been deployed to https://90ac4ce6.helpdot.pages.dev ⚡️ Updated articles: |
Summary
Copilot-Access.mdnavigation paths from Account > Security to Account > Copilot to reflect Copilot becoming a dedicated top-level menu itemNavigating-New-Expensify.mdto list Copilot as its own item under the Account tab, separate from SecurityRelated PR: #89422
Test plan
Copilot-Access.mdreference Account > Copilot (not Account > Security)Navigating-New-Expensify.mdlists Copilot as a separate item from Security under Account