Skip to content

docs: standardize workflows-overview structure and add missing components#1677

Merged
serikjensen merged 2 commits intomainfrom
docs/workflows-overview-organization
May 7, 2026
Merged

docs: standardize workflows-overview structure and add missing components#1677
serikjensen merged 2 commits intomainfrom
docs/workflows-overview-organization

Conversation

@serikjensen
Copy link
Copy Markdown
Member

@serikjensen serikjensen commented May 5, 2026

Summary

Standardizes every doc under docs/workflows-overview/ against the company-onboarding.md template so partners can scan them the same way:

  • Available Subcomponents up top, anchor-linked. Each workflow now leads with a single ### Available Subcomponents list whose links jump to per-component sections with consistent Props / Events tables.
  • Documented missing components. Audited src/index.ts against existing docs and filled in:
    • CompanyOnboarding.Industry, CompanyOnboarding.OnboardingOverview, CompanyOnboarding.DocumentList, CompanyOnboarding.SignatureForm
    • EmployeeOnboarding.EmploymentEligibility
    • EmployeeManagement.EmployeeList, EmployeeManagement.WorkAddress
    • Payroll.PayrollExecutionFlow, Payroll.RecoveryCases, and corrected Payroll.PayrollBlockerPayroll.PayrollBlockerList
  • New information-requests.md workflow doc covering InformationRequests.InformationRequestsFlow, InformationRequestList, and InformationRequestForm (previously listed in the overview index with no dedicated page).
  • Cross-references for the shared payroll execution phase. off-cycle-payroll.md, transition-payroll.md, and dismissal-payroll.md now link to Payroll.PayrollExecutionFlow in run-payroll.md instead of duplicating it.
  • Event-name consistency. employee-dashboard.md event tables and the example handler now use componentEvents.* constants (e.g. EMPLOYEE_HOME_ADDRESS) instead of the lowercase string literals it had drifted to.
  • Backwards-compatible export fix. Added StateTaxesForm / StateTaxesList to src/components/Company/exports/companyOnboarding.ts so they're available under CompanyOnboarding.* like every other subcomponent. Existing Company.StateTaxesForm / Company.StateTaxesList consumers are unaffected (the legacy Company.* namespace still re-exports the same components), so the docs no longer need the "(legacy Company.* only)" annotation.

No runtime behavior changes outside the additive CompanyOnboarding re-exports.

Test plan

  • npm run build succeeds (lint-staged ran the build during commit; verify in CI).
  • Skim each updated workflow doc in the rendered docs site / preview and confirm anchor links resolve to the right subcomponent sections.
  • Confirm import { CompanyOnboarding } from '@gusto/embedded-react-sdk' exposes CompanyOnboarding.StateTaxesForm and CompanyOnboarding.StateTaxesList, and that Company.StateTaxesForm / Company.StateTaxesList still resolve to the same components.

Made with Cursor

…ents

Aligns all workflow docs with the company-onboarding template (Available
Subcomponents up top, anchor-linked, consistent Props/Events tables).
Adds previously undocumented components and a dedicated Information
Requests workflow doc. Also exports StateTaxesForm and StateTaxesList
under the CompanyOnboarding namespace so docs can drop the legacy-only
note (Company.* legacy paths still work).

Co-authored-by: Cursor <cursoragent@cursor.com>
@gusto-fresh-eyes
Copy link
Copy Markdown

gusto-fresh-eyes Bot commented May 5, 2026

No issues found

Generated by Fresh Eyes Reviewer | Share feedback in #ai-code-reviews

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is so much more clear now! Is there any way we can also mention the recommended subcomponents for other events? Things like the employee details, compensation, etc. Or just make it very clear when there isn't a standalone component you can just use

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👋 is this feedback specific to employee dashboard? if so, this should get fleshed out more completely as we round out the remainder of the steady state components and get them documented!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me!

@serikjensen serikjensen enabled auto-merge (squash) May 7, 2026 17:08
@serikjensen serikjensen merged commit 81e25d8 into main May 7, 2026
20 checks passed
@serikjensen serikjensen deleted the docs/workflows-overview-organization branch May 7, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants