[No QA] Update help docs for Gusto HR integration in New Expensify#91754
Conversation
- New article: Connect Gusto to Expensify (new-expensify/connections/Gusto.md) covering HR feature enablement, OAuth connection, approval mode configuration, sync results, Members page sync status, manual re-sync, and disconnection. - Update Add-Approvals.md with a callout explaining that when Gusto is connected, the Approvals toggle is locked on and managed from the HR page. Co-authored-by: Stephanie Elliott <stephanieelliott@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
|
The The Cloudflare asset upload API returned a
The files were successfully uploaded (1523/1523), but the API returned a malformed response afterward, causing wrangler to throw a Fix: Re-run the failed |
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
@greg-schroeder 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] |
|
Both failing checks are unrelated to this PR's code changes — no fix is needed.
Fix: Re-run the failed |
|
I'm still working on this @yuwenmemon - I will tap you when it's ready for your review! |
This comment has been minimized.
This comment has been minimized.
|
The spellcheck failure was caused by a typo ( The previous |
This comment has been minimized.
This comment has been minimized.
|
All checks are now passing ✅ — the |
|
All set @yuwenmemon! |
HelpDot Documentation ReviewOverall AssessmentThis PR adds a comprehensive new article for the Gusto integration and a well-placed cross-reference in the existing Approvals article. The new article is well-structured, covers the full lifecycle (connect, configure, view, disconnect), and follows governance rules closely. A few minor phrasing issues were identified. Scores Summary
Key Findings
Recommendations
Files Reviewed
|
|
@MelvinBot can you update your PR to incorporate the review here: #91754 (comment) |
|
Thanks for the review feedback! I've looked at the two recommendations from the HelpDot review:
I can only make code changes when directed by the PR author, an assignee, or an existing co-author. To have me implement this, the PR assignee or author should reply with a request tagging me. |
|
@MelvinBot can you update your PR to incorporate the review here: #91754 (comment) |
Change "and at the bottom on mobile" to "on the bottom on mobile" to match the standard help site navigation instruction format. Co-authored-by: Stephanie Elliott <stephanieelliott@users.noreply.github.com>
|
Addressed the HelpDot review feedback: Fixed: Navigation phrasing in step 1 — changed "and at the bottom on mobile" → "on the bottom on mobile" to match the standard governance wording used across all other help articles. Not changed: "In the left menu" phrasing — this is already the established convention used in other articles (e.g., |
This comment has been minimized.
This comment has been minimized.
HelpDot Documentation ReviewOverall AssessmentThis PR introduces a new Gusto integration article and adds a cross-reference note to the existing Add-Approvals article. The new article is well-structured, follows governance standards closely, and provides clear procedural guidance. The cross-link addition to Add-Approvals is minimal and well-placed. Overall, this is a strong submission with only minor items to consider. Scores Summary
Key Findings
Recommendations
Files Reviewed
Review based on governance files: HELPSITE_NAMING_CONVENTIONS.md, HELP_AUTHORING_GUIDELINES.md, and TEMPLATE.md. |
…sify.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
|
A preview of your ExpensifyHelp changes have been deployed to https://4568537e.helpdot.pages.dev ⚡️ Updated articles: |
|
@yuwenmemon looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
|
Tests were passing from what I saw. And the latest commit confirms that. |
|
🚀 Deployed to staging by https://github.com/yuwenmemon in version: 9.3.89-1 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
Adds help site documentation for the Gusto HR integration in New Expensify, covering all three releases scoped in Expensify/Expensify#615695:
New article:
docs/articles/new-expensify/connections/Gusto.mdUpdated article:
docs/articles/new-expensify/workspaces/Add-Approvals.mdFixed Issues
$ https://github.com/Expensify/Expensify/issues/615695