Skip to content

feat(contractor-onboarding) - add KWT, Oman, Qatar, Bahrain edge cases#729

Merged
gabrielseco merged 10 commits intomainfrom
pbyr-3461-add-more-edge-cases-countries-to-the-sdk
Feb 19, 2026
Merged

feat(contractor-onboarding) - add KWT, Oman, Qatar, Bahrain edge cases#729
gabrielseco merged 10 commits intomainfrom
pbyr-3461-add-more-edge-cases-countries-to-the-sdk

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented Feb 4, 2026

Note

Medium Risk
Touches onboarding form field names and payload mapping (saudi_nationality_statusnationality_status) and adjusts when/where countries are fetched, which can break submissions or labels if any downstream expectations weren’t updated.

Overview
Adds a shared useCountries React Query hook and standardizes country-fetch mocking via a new countriesMock fixture plus a default MSW */v1/countries handler, removing per-test country handlers.

Extends contractor onboarding “nationality status” handling beyond Saudi Arabia by renaming the field to nationality_status, persisting it for SAU/KWT/OMN/QAT/BHR, and making the UI copy dynamic using the selected country name. The contractor onboarding flow now also loads the full countries list up-front to derive countryName for this copy.

Onboarding flow country selection is refactored to use the new shared useCountries hook while preserving the EOR onboarding filter for its select options.

Written by Cursor Bugbot for commit 8aaeb89. This will update automatically on new commits. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 187.56 kB 186.65 kB +912 B (+0.5%) 🔴
Total (raw) 501.53 kB 499.34 kB +2.19 kB (+0.4%) 🔴
CSS (gzip) 19.37 kB 19.37 kB 0 B (0%) 🟢
CSS (raw) 101.83 kB 101.83 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 187.56 kB / 200 kB (93.8%)
  • ✅ Total raw: 501.53 kB / 550 kB (91.2%)
  • ✅ CSS gzipped: 19.37 kB / 25 kB (77.5%)

Largest Files (Top 5)

  1. chunk-ZSJ6YADQ.js - 12.43 kB (0 B (0%))
  2. styles.css - 9.69 kB (0 B (0%))
  3. index.css - 9.69 kB (0 B (0%))
  4. index.js - 5.87 kB (+8 B (+0.1%))
  5. chunk-BYX2YS2B.js - 5.61 kB (0 B (0%))
View All Files (284 total)
File Size (gzip) Change
chunk-ZSJ6YADQ.js 12.43 kB 0 B (0%)
styles.css 9.69 kB 0 B (0%)
index.css 9.69 kB 0 B (0%)
index.js 5.87 kB +8 B (+0.1%)
chunk-BYX2YS2B.js 5.61 kB 0 B (0%)
chunk-BBOEQU3O.js 4.07 kB 0 B (0%)
chunk-XGLS7WMN.js 3.82 kB new
chunk-T7O3YYOG.js 3.29 kB 0 B (0%)
chunk-N4MOUCQT.js 3.29 kB new
chunk-KQK572PW.js 3.21 kB new

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

Deploy preview for remote-flows-example-app ready!

✅ Preview
https://remote-flows-example-ae1hcxv5c-remotecom.vercel.app

Built with commit 8aaeb89.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

Deploy preview for remote-flows ready!

✅ Preview
https://remote-flows-cpiawbpw4-remotecom.vercel.app

Built with commit 8aaeb89.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

npm warn exec The following package was not found and will be installed: tsx@4.21.0

📊 Coverage Report

⚠️ Coverage decreased

Metric Current Previous Change Status
Lines 87.94% 87.89% +0.05% 🟢
Statements 87.59% 87.54% +0.05% 🟢
Functions 83.75% 83.72% +0.03% 🟢
Branches 79.09% 79.64% -0.55% 🔴

Detailed Breakdown

Lines Coverage
  • Covered: 3033 / 3449
  • Coverage: 87.94%
  • Change: +0.05% (21 lines)
Statements Coverage
  • Covered: 3091 / 3529
  • Coverage: 87.59%
  • Change: +0.05% (22 statements)
Functions Coverage
  • Covered: 866 / 1034
  • Coverage: 83.75%
  • Change: +0.03% (7 functions)
Branches Coverage
  • Covered: 1914 / 2420
  • Coverage: 79.09%
  • Change: -0.55% (9 branches)

✅ Coverage check passed

@gabrielseco
Copy link
Copy Markdown
Collaborator Author

bugbot run verbose=true

@cursor
Copy link
Copy Markdown

cursor Bot commented Feb 5, 2026

Bugbot request id: serverGenReqId_6275302f-3d41-44a2-9ffa-e2a3ac99a3a5

@cursor
Copy link
Copy Markdown

cursor Bot commented Feb 5, 2026

Skipping Bugbot: Unable to authenticate your request. Please make sure Bugbot is properly installed and configured for this repository.

@gabrielseco gabrielseco changed the title feat(contractor-onboarding) - add KWT, Oman, Qatar edge cases feat(contractor-onboarding) - add KWT, Oman, Qatar, Bahrain edge cases Feb 10, 2026
@gabrielseco gabrielseco marked this pull request as ready for review February 18, 2026 17:15
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread src/flows/ContractorOnboarding/jsfModify.tsx Outdated
@gabrielseco gabrielseco merged commit 48fed5b into main Feb 19, 2026
7 checks passed
@gabrielseco gabrielseco deleted the pbyr-3461-add-more-edge-cases-countries-to-the-sdk branch February 19, 2026 09:07
@gabrielseco gabrielseco mentioned this pull request Feb 20, 2026
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.

2 participants