feat(contractor-onboarding) - add KWT, Oman, Qatar, Bahrain edge cases#729
Conversation
📦 Bundle Size Report
Size Limits
Largest Files (Top 5)
View All Files (284 total)
✅ Bundle size check passed |
|
Deploy preview for remote-flows-example-app ready! ✅ Preview Built with commit 8aaeb89. |
|
Deploy preview for remote-flows ready! ✅ Preview Built with commit 8aaeb89. |
|
npm warn exec The following package was not found and will be installed: tsx@4.21.0 📊 Coverage Report
|
| 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
|
bugbot run verbose=true |
|
Bugbot request id: serverGenReqId_6275302f-3d41-44a2-9ffa-e2a3ac99a3a5 |
|
Skipping Bugbot: Unable to authenticate your request. Please make sure Bugbot is properly installed and configured for this repository. |
…-edge-cases-countries-to-the-sdk
There was a problem hiding this comment.
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.
Note
Medium Risk
Touches onboarding form field names and payload mapping (
saudi_nationality_status→nationality_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
useCountriesReact Query hook and standardizes country-fetch mocking via a newcountriesMockfixture plus a default MSW*/v1/countrieshandler, removing per-test country handlers.Extends contractor onboarding “nationality status” handling beyond Saudi Arabia by renaming the field to
nationality_status, persisting it forSAU/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 derivecountryNamefor this copy.Onboarding flow country selection is refactored to use the new shared
useCountrieshook 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.