Skip to content

fix(contractor-onboarding) - fix disabled button state#1022

Merged
gabrielseco merged 2 commits into
mainfrom
fix-disabled-state-contract-review
May 15, 2026
Merged

fix(contractor-onboarding) - fix disabled button state#1022
gabrielseco merged 2 commits into
mainfrom
fix-disabled-state-contract-review

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented May 15, 2026

Note

Low Risk
Low risk: small UI-state change plus dependency patch bumps; main impact is button interactivity during form submission.

Overview
Fixes the contractor onboarding contract review CTA so it is disabled during form submission by combining props.disabled with contractorOnboardingBag.isSubmitting, and ensures disabled is applied consistently on the CustomButton trigger.

Updates package-lock.json with patch-level dependency bumps (notably @tanstack/react-query, msw, and oxfmt) and related lockfile metadata changes.

Reviewed by Cursor Bugbot for commit 3b523ff. Bugbot is set up for automated code reviews on this repo. Configure here.

@gabrielseco gabrielseco self-assigned this May 15, 2026
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.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 54031af. Configure here.

Comment thread src/flows/ContractorOnboarding/components/ContractReviewButton.tsx
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 211.76 kB 211.73 kB +38 B (+0.0%) 🔴
Total (raw) 574 kB 573.95 kB +48 B (+0.0%) 🔴
CSS (gzip) 20.86 kB 20.86 kB 0 B (0%) 🟢
CSS (raw) 108.83 kB 108.83 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 211.76 kB / 250 kB (84.7%)
  • ✅ Total raw: 574 kB / 600 kB (95.7%)
  • ✅ CSS gzipped: 20.86 kB / 25 kB (83.4%)

Largest Files (Top 5)

  1. chunk-5BU3CD2P.js - 14 kB (0 B (0%))
  2. styles.css - 10.43 kB (0 B (0%))
  3. index.css - 10.43 kB (0 B (0%))
  4. index.js - 6.11 kB (+1 B (+0.0%))
  5. chunk-ER46UFIU.js - 5.89 kB (0 B (0%))
View All Files (312 total)
File Size (gzip) Change
chunk-5BU3CD2P.js 14 kB 0 B (0%)
styles.css 10.43 kB 0 B (0%)
index.css 10.43 kB 0 B (0%)
index.js 6.11 kB +1 B (+0.0%)
chunk-ER46UFIU.js 5.89 kB 0 B (0%)
chunk-U3PQEGSV.js 5.81 kB 0 B (0%)
chunk-JXAAON7I.js 5.26 kB 0 B (0%)
chunk-PMAIML44.js 4.62 kB 0 B (0%)
chunk-T35SZZ74.js 4.45 kB new
chunk-3PKL3V6F.js 4.06 kB 0 B (0%)

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2026

📊 Coverage Report

✅ Coverage increased! 🎉

Metric Current Previous Change Status
Lines 90.05% 90.05% 0%
Statements 89.72% 89.72% 0%
Functions 87.73% 87.73% 0%
Branches 80.09% 80.08% +0.01% 🟢

Detailed Breakdown

Lines Coverage
  • Covered: 3694 / 4102
  • Coverage: 90.05%
  • Change: 0% (1 lines)
Statements Coverage
  • Covered: 3754 / 4184
  • Coverage: 89.72%
  • Change: 0% (1 statements)
Functions Coverage
  • Covered: 1001 / 1141
  • Coverage: 87.73%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 2301 / 2873
  • Coverage: 80.09%
  • Change: +0.01% (2 branches)

✅ Coverage check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2026

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-6rghrwdf3-remotecom.vercel.app
Latest Commit:3b523ff
Inspect:View deployment

Deployed with vercel-action

@gabrielseco gabrielseco merged commit 53445e6 into main May 15, 2026
10 checks passed
@gabrielseco gabrielseco deleted the fix-disabled-state-contract-review branch May 15, 2026 13:56
@gabrielseco gabrielseco mentioned this pull request May 15, 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