[No QA] Improve Copilot feature documentation and discoverability#91786
[No QA] Improve Copilot feature documentation and discoverability#91786MelvinBot wants to merge 20 commits into
Conversation
Co-authored-by: Stephanie Elliott <stephanieelliott@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
HelpDot Documentation ReviewOverall AssessmentThis PR improves discoverability across four Copilot and Delegate articles by expanding keywords, adding comparison tables, common use cases, and cross-reference FAQ entries. The additions are substantively helpful for users who confuse Copilot and Vacation Delegate features. However, several structural and formatting issues conflict with the HelpDot governance rules. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
HelpDot Documentation ReviewOverall AssessmentThis PR adds cross-linking between Copilot and Vacation Delegate articles across both Expensify Classic and New Expensify, introduces comparison tables, expands keywords for better search coverage, and adds a new FAQ entry to the Delegate articles. The intent is sound -- helping users distinguish between Copilots and Vacation Delegates is a genuine user need. However, several structural and formatting issues must be resolved before merging. Scores Summary
Key FindingsCritical issues that must be fixed:
Positive aspects:
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| | **Best for** | Executive assistants, accountants, or anyone who regularly helps manage your expenses | Covering approvals during vacation or leave | | ||
|
|
||
| If you need someone to temporarily handle approvals while you're out of office, see [Delegate When Out of Office](/articles/expensify-classic/copilots-and-delegates/Delegate-when-out-of-office) instead. | ||
| If you need someone to temporarily handle approvals while you're out of office, see [Delegate When Out of Office](/articles/expensify-classic/copilots-and-delegates/Delegate-when-out-of-office) instead. |
There was a problem hiding this comment.
Dupe line here. 85 & 86 are the same.
| ## I need someone to permanently manage my expenses, not just approvals while I'm away. What should I use? | ||
|
|
||
| A Vacation Delegate is designed for **temporary** approval coverage. If you need someone to have **ongoing access** to your account -- for example, an executive assistant or accountant who regularly manages your expenses -- use a [Copilot](https://help.expensify.com/articles/expensify-classic/copilots-and-delegates/Assign-or-remove-a-Copilot) instead. | ||
| A Vacation Delegate is designed for **temporary** approval coverage. If you need someone to have **ongoing access** to your account -- for example, an executive assistant or accountant who regularly manages your expenses -- use a [Copilot](/articles/expensify-classic/copilots-and-delegates/Assign-or-remove-a-Copilot) instead. |
There was a problem hiding this comment.
Still seems to be a dupe line here.
Updated the content to clarify Copilot access and its functionalities.
|
@MelvinBot can you run the |
|
🤖 QA review of the updated files — found 2 broken links, a duplicated line, a missing redirect, and a front-matter format issue. Details below. 🔴 Blockers1. Broken link — classic 2. Broken link — new 3. Duplicated answer line — classic 4. Missing redirect for the deleted 🟡 Should fix5. Malformed front matter — blank line after the opening
🔵 Lower-confidence — navigation steps to verify against the live UIThe navigation paths to the Copilot settings differ across the articles and I couldn't verify which is current from the docs alone — worth a manual check:
Please confirm the real in-product location so all four articles describe the same path. I'm the author of this PR, so reply with |
Resolve modify/delete conflict by keeping the split Copilot docs and applying main's Limited Access updates and Add a copilot button label changes. Co-authored-by: Cursor <cursoragent@cursor.com>
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
@Gonals @kadiealexander 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 |
Summary
Related to Expensify/Expensify#637958
Test plan