[No QA] Add search keyword synonyms for help site discoverability#91789
Open
MelvinBot wants to merge 1 commit into
Open
[No QA] Add search keyword synonyms for help site discoverability#91789MelvinBot wants to merge 1 commit into
MelvinBot wants to merge 1 commit into
Conversation
Co-authored-by: Stephanie Elliott <stephanieelliott@users.noreply.github.com>
github-actions
Bot
changed the title
Contributor
Contributor
HelpDot Documentation ReviewOverall AssessmentThis PR adds keyword synonyms to the Scores Summary
Key FindingsPositive aspects:
Minor issues:
Recommendations
Files Reviewed
Note: This PR contains metadata-only changes (keywords), so inline comments were not necessary. All feedback is captured in this summary. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| title: Close Your Expensify Account | ||
| description: Learn how to close your Expensify account. | ||
| keywords: [New Expensify, close account, delete Expensify account, delete account, remove user, deactivate account, cancel account, cancel Expensify, terminate, how to delete Expensify, how to cancel Expensify] | ||
| keywords: [New Expensify, close account, delete Expensify account, delete account, remove user, deactivate account, cancel account, cancel Expensify, terminate, how to delete Expensify, how to cancel Expensify, remove account, close account, terminate account, delete Expensify, how to delete, how to cancel] |
Contributor
There was a problem hiding this comment.
Duplicate keyword: close account appears twice in this keywords array (positions 2 and 13). Remove the duplicate to keep metadata clean.
| title: Close Your Expensify Account | ||
| description: Learn how to close your Expensify account. | ||
| keywords: [Expensify Classic, close account, delete account, cancel account, delete Expensify account, cancel Expensify, deactivate account, how to delete Expensify, how to cancel Expensify] | ||
| keywords: [Expensify Classic, close account, delete account, cancel account, delete Expensify account, cancel Expensify, deactivate account, how to delete Expensify, how to cancel Expensify, remove account, terminate account, delete Expensify, how to delete, how to cancel, remove user] |
Contributor
There was a problem hiding this comment.
Terminology violation: Per HELPSITE_NAMING_CONVENTIONS.md, use "member" instead of "user." The keyword remove user should be remove member.
| title: Close Your Expensify Account | ||
| description: Learn how to close your Expensify account. | ||
| keywords: [New Expensify, close account, delete Expensify account, delete account, remove user, deactivate account, cancel account, cancel Expensify, terminate, how to delete Expensify, how to cancel Expensify] | ||
| keywords: [New Expensify, close account, delete Expensify account, delete account, remove user, deactivate account, cancel account, cancel Expensify, terminate, how to delete Expensify, how to cancel Expensify, remove account, close account, terminate account, delete Expensify, how to delete, how to cancel] |
Contributor
There was a problem hiding this comment.
Terminology violation: Per HELPSITE_NAMING_CONVENTIONS.md, use "member" instead of "user." The keyword remove user should be remove member.
| title: Create an Expense | ||
| description: Learn how to create and submit expenses in Expensify using SmartScan, manual entry, distance tracking, or time expenses. | ||
| keywords: [create expense, submit expense, SmartScan, manual expense, distance expense, time expense, create time expense, log time, track hours, expense report, submit to workspace, submit to individual, split expense, scan receipts, bulk upload, GPS, GPS tracking, GPS mileage, Proofpoint, email forwarding, receipt forwarding] | ||
| keywords: [create expense, submit expense, SmartScan, manual expense, distance expense, time expense, create time expense, log time, track hours, expense report, submit to workspace, submit to individual, split expense, scan receipts, bulk upload, GPS, GPS tracking, GPS mileage, Proofpoint, email forwarding, receipt forwarding, email receipt, forward receipt, forward email receipt, email receipts@expensify.com, scan receipt, receipt email] |
Contributor
There was a problem hiding this comment.
AI readiness / metadata consistency: This keywords array is missing New Expensify as the first keyword. All other New Expensify articles include it (e.g., [New Expensify, create expense, ...]). Adding it improves retrieval accuracy and matches the pattern across the help site.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 49e9ae5131
ℹ️ 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".
| title: Close Your Expensify Account | ||
| description: Learn how to close your Expensify account. | ||
| keywords: [Expensify Classic, close account, delete account, cancel account, delete Expensify account, cancel Expensify, deactivate account, how to delete Expensify, how to cancel Expensify] | ||
| keywords: [Expensify Classic, close account, delete account, cancel account, delete Expensify account, cancel Expensify, deactivate account, how to delete Expensify, how to cancel Expensify, remove account, terminate account, delete Expensify, how to delete, how to cancel, remove user] |
There was a problem hiding this comment.
When a Classic workspace admin searches for “remove user”, this keyword now makes the account-closure article match even though the article’s own scope says it does not cover removing members from a domain/workspace, and the Classic workspace member-removal flow lives in docs/articles/expensify-classic/workspaces/Remove-Members.md. This will steer member-offboarding searches toward instructions for permanently closing an individual account instead of removing a workspace member.
Useful? React with 👍 / 👎.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
keywordsfrontmatter array across 15 help articles (both New Expensify and Expensify Classic) to improve search discoverabilityRelated: Expensify/Expensify#637958
Test plan
keywordsarrays in frontmatter