Skip to content

fix(france) - fix conditionals by using inputType#1175

Merged
gabrielseco merged 1 commit into
mainfrom
fix-money-type-for-france
Jul 21, 2026
Merged

fix(france) - fix conditionals by using inputType#1175
gabrielseco merged 1 commit into
mainfrom
fix-money-type-for-france

Conversation

@gabrielseco

@gabrielseco gabrielseco commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

The problem was when you select in local because its where we're getting the correct schema, you select contract_duration_type: indefinite and then you type annual_gross_salary, the field wasn't getting marked correctly as a money value which made some hidden fields to not appear

https://www.loom.com/share/76ac3c99b0c5448cbdbb645743eb0b97

@gabrielseco gabrielseco self-assigned this Jul 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 242.12 kB 242.12 kB -4 B (-0.0%) 🟢
Total (raw) 648.78 kB 648.74 kB +41 B (+0.0%) 🔴
CSS (gzip) 21.45 kB 21.45 kB 0 B (0%) 🟢
CSS (raw) 110.5 kB 110.5 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 242.12 kB / 250 kB (96.8%)
  • ✅ Total raw: 648.78 kB / 650 kB (99.8%)
  • ✅ CSS gzipped: 21.45 kB / 25 kB (85.8%)

Largest Files (Top 5)

  1. chunk-Y3CJRZLA.js - 14 kB (0 B (0%))
  2. styles.css - 10.73 kB (0 B (0%))
  3. index.css - 10.73 kB (0 B (0%))
  4. index.js - 6.49 kB (-3 B (-0.0%))
  5. chunk-TG57ZBOY.js - 6.44 kB (new)
View All Files (381 total)
File Size (gzip) Change
chunk-Y3CJRZLA.js 14 kB 0 B (0%)
styles.css 10.73 kB 0 B (0%)
index.css 10.73 kB 0 B (0%)
index.js 6.49 kB -3 B (-0.0%)
chunk-TG57ZBOY.js 6.44 kB new
chunk-AS7SJDU5.js 6.4 kB 0 B (0%)
chunk-G2LZFGZA.js 5.31 kB 0 B (0%)
chunk-GX3ULGDN.js 4.86 kB new
chunk-CG6HX6UO.js 4.76 kB 0 B (0%)
chunk-LSC6NTXD.js 3.89 kB 0 B (0%)

✅ Bundle size check passed

@github-actions

Copy link
Copy Markdown
Contributor

📊 Coverage Report

✅ Coverage increased! 🎉

Metric Current Previous Change Status
Lines 83.46% 83.37% +0.08% 🟢
Statements 82.83% 82.75% +0.08% 🟢
Functions 81.40% 81.32% +0.08% 🟢
Branches 75.59% 75.52% +0.08% 🟢

Detailed Breakdown

Lines Coverage
  • Covered: 3975 / 4763
  • Coverage: 83.46%
  • Change: +0.08% (4 lines)
Statements Coverage
  • Covered: 4039 / 4876
  • Coverage: 82.83%
  • Change: +0.08% (4 statements)
Functions Coverage
  • Covered: 1072 / 1317
  • Coverage: 81.40%
  • Change: +0.08% (1 functions)
Branches Coverage
  • Covered: 2478 / 3278
  • Coverage: 75.59%
  • Change: +0.08% (4 branches)

✅ Coverage check passed

@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for adp-cost-calculator ready!

Project:adp-cost-calculator
Status: ✅  Deploy successful!
Preview URL:https://adp-cost-calculator-i94v48fp9-remotecom.vercel.app
Latest Commit:1889c16

Deployed with vercel-action

@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-64sxnoa73-remotecom.vercel.app
Latest Commit:1889c16

Deployed with vercel-action

@gabrielseco
gabrielseco merged commit 98253fd into main Jul 21, 2026
10 checks passed
@gabrielseco
gabrielseco deleted the fix-money-type-for-france branch July 21, 2026 08:53
@gabrielseco gabrielseco mentioned this pull request Jul 23, 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