Skip to content

feat(contractor): add self-onboarding OnboardingSummary component#2300

Merged
serikjensen merged 5 commits into
mainfrom
feat/contractor-onboarding-summary
Jun 30, 2026
Merged

feat(contractor): add self-onboarding OnboardingSummary component#2300
serikjensen merged 5 commits into
mainfrom
feat/contractor-onboarding-summary

Conversation

@serikjensen

@serikjensen serikjensen commented Jun 29, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds ContractorOnboarding.OnboardingSummary, a self-onboarding completion screen shown at the end of the contractor self-onboarding flow. It mirrors the employee OnboardingSummary self view and derives its copy/layout/design from the sdk-app ContractorOnboardingComplete design reference (centered "You're all set!" heading, supporting body, single secondary "Done" button).
  • Emits a new contractor/selfOnboarding/done event (CONTRACTOR_SELF_ONBOARDING_DONE) with { contractorId } when the contractor acknowledges submission, pairing with the existing CONTRACTOR_SELF_ONBOARDING_START.
  • Self-onboarding only — no isAdmin/admin view, since no contractor onboarding summary existed previously.

Changes

  • src/components/Contractor/OnboardingSummary/ — new component + barrel export, with TSDoc on the export and props.
  • src/i18n/en/Contractor.OnboardingSummary.jsonsubtitle / description / doneCta copy.
  • src/shared/constants.ts — new CONTRACTOR_SELF_ONBOARDING_DONE event.
  • src/components/Contractor/exports/contractorOnboarding.ts — exports OnboardingSummary + OnboardingSummaryProps.
  • Regenerated derived files: src/types/i18next.d.ts, .reports/embedded-react-sdk.api.md, sdk-app/src/generated-registry-data.ts.

Out of scope

  • No flow wiring: there is no shipped contractor SelfOnboardingFlow in the SDK, so no *Contextual wrapper or state-machine step is added. The component is a standalone export partners place at the end of their self-onboarding wizard.

Test plan

  • tsc --noEmit passes
  • npm run lint:check — 0 errors
  • npm run test -- --run src/components/Contractor — 102/102 pass
  • API Extractor report regenerated to include the new export
  • Visual verification of the completion screen in a self-onboarding context
Screenshot 2026-06-29 at 10 24 50 AM

Made with Cursor

Adds ContractorOnboarding.OnboardingSummary, a self-onboarding completion
screen shown at the end of the contractor self-onboarding flow. Emits
contractor/selfOnboarding/done when the contractor acknowledges submission.

Co-authored-by: Cursor <cursoragent@cursor.com>
@serikjensen serikjensen requested a review from a team as a code owner June 29, 2026 16:27
@serikjensen

Copy link
Copy Markdown
Member Author

@cursoragent bring this up to date and resolve merge conflicts

@serikjensen serikjensen enabled auto-merge June 30, 2026 18:43
@cursor

cursor Bot commented Jun 30, 2026

Copy link
Copy Markdown

Summary

Successfully resolved merge conflicts and brought the branch up to date with main.

Conflicts resolved:

  • docs/reference/index.md: Merged both Contractor.DocumentsList (from main) and Contractor.OnboardingSummary (from this branch) into the dictionary documentation in alphabetical order
  • src/types/i18next.d.ts: Added both new contractor namespace entries to the resources interface

Changes from main merged in:

  • New Contractor.DocumentsList component and its supporting infrastructure
  • Various documentation and API reference updates
  • Package updates

The branch is now up to date, all conflicts are resolved, and the changes have been pushed to the remote repository. The PR should now be ready for review without any merge conflicts.

Open in Web Open in Cursor 

Resolved conflicts in:
- docs/reference/index.md
- src/types/i18next.d.ts

Both branches added new contractor components:
- main added Contractor.DocumentsList
- This branch added Contractor.OnboardingSummary
Merged both additions into the resources list.

Co-authored-by: Steve Jensen <stephen.erik.jensen@gmail.com>
Comment on lines +19 to +25
/**
* Confirmation screen shown at the end of the contractor self-onboarding flow.
* Lets the contractor know their information has been submitted and emits
* `contractor/selfOnboarding/done` when they acknowledge it.
*
* @public
*/

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.

Can you tell your cursor agent to emit a proper @events section and table with all events emitted and their payloads?

@serikjensen

Copy link
Copy Markdown
Member Author

@cursoragent could you resolve feedback from marie above around a proper @events section. We also need to update, resolve any conflicts. also fix any failures with ci causing issues

@cursor

cursor Bot commented Jun 30, 2026

Copy link
Copy Markdown

Perfect! All tasks completed successfully:

Summary

Resolved Marie's feedback: Added proper @events section to the OnboardingSummary TSDoc with a table documenting:

  • Event name: contractor/selfOnboarding/done
  • Description: Fired when the contractor acknowledges completion and clicks the Done button
  • Data payload: { contractorId: string }

Updated and resolved conflicts: Successfully merged latest main branch, resolving conflicts in:

  • docs/reference/index.md - merged both Contractor.DocumentsList and Contractor.OnboardingSummary
  • src/types/i18next.d.ts - included both new contractor namespace entries

Fixed CI issues:

  • Build: ✅ Passes successfully
  • Lint: ✅ 0 errors (26 pre-existing warnings unrelated to this PR)
  • Tests: ✅ All 106 Contractor tests pass
  • Derive: ✅ Successfully regenerated all derived files

The PR is now ready for review with:

  • Proper event documentation in TSDoc format matching the SDK's conventions
  • No merge conflicts
  • All checks passing locally
  • Changes committed and pushed to the branch
Open in Web Open in Cursor 

cursoragent and others added 2 commits June 30, 2026 20:00
- Add proper @events table documenting the contractor/selfOnboarding/done event
- Include event name, description, and data payload format
- Regenerate i18next.d.ts to include ContractorDocumentsList from main

Resolves feedback from @mariechatfield
@serikjensen serikjensen added this pull request to the merge queue Jun 30, 2026
Merged via the queue into main with commit 8053855 Jun 30, 2026
32 checks passed
@serikjensen serikjensen deleted the feat/contractor-onboarding-summary branch June 30, 2026 20:20
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.

4 participants