Skip to content

feat(playground) - Add initial values to playground#1180

Merged
gabrielseco merged 3 commits into
mainfrom
add-initial-values-to-playground
Jul 22, 2026
Merged

feat(playground) - Add initial values to playground#1180
gabrielseco merged 3 commits into
mainfrom
add-initial-values-to-playground

Conversation

@gabrielseco

Copy link
Copy Markdown
Collaborator

Add initialValues to playground

The issue was that money fields were being converted to cents twice:

1. createHeadlessForm was calling convertToCents/parseFloat on money fields
2. The field transformations (transformValueToAPI) were calling convertToCents again

This caused:
- Input: 50000 EUR
- First conversion: 5000000 cents (in createHeadlessForm)
- Second conversion: 500000000 cents (in field transformations)
- Schema computation: 500000000 * 0.05 = 25000000 cents
- Display: 25000000 / 100 = 250000 EUR (wrong!)

Fix: Remove money field processing from createHeadlessForm entirely.
Let the field transformations handle the conversion as they should.

Now:
- Input: 50000 EUR
- Field transformation: convertToCents(50000) = 5000000 cents
- Schema computation: 5000000 * 0.05 = 250000 cents
- Display: 250000 / 100 = 2500 EUR (correct!)
Comment thread package.json
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 256.63 kB 256.49 kB +148 B (+0.1%) 🔴
Total (raw) 704.03 kB 703.73 kB +301 B (+0.0%) 🔴
CSS (gzip) 21.69 kB 21.69 kB 0 B (0%) 🟢
CSS (raw) 112.66 kB 112.66 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 256.63 kB / 350 kB (73.3%)
  • ✅ Total raw: 704.03 kB / 850 kB (82.8%)
  • ✅ CSS gzipped: 21.69 kB / 25 kB (86.7%)

Largest Files (Top 5)

  1. chunk-Y3CJRZLA.js - 14 kB (0 B (0%))
  2. styles.css - 10.84 kB (0 B (0%))
  3. index.css - 10.84 kB (0 B (0%))
  4. chunk-AVIZTG4K.js - 8.54 kB (0 B (0%))
  5. index.js - 6.56 kB (+2 B (+0.0%))
View All Files (400 total)
File Size (gzip) Change
chunk-Y3CJRZLA.js 14 kB 0 B (0%)
styles.css 10.84 kB 0 B (0%)
index.css 10.84 kB 0 B (0%)
chunk-AVIZTG4K.js 8.54 kB 0 B (0%)
index.js 6.56 kB +2 B (+0.0%)
chunk-746NEWTS.js 6.44 kB new
chunk-AS7SJDU5.js 6.4 kB 0 B (0%)
chunk-G2LZFGZA.js 5.31 kB 0 B (0%)
chunk-ERDNYHQO.js 4.86 kB 0 B (0%)
chunk-43FMKELR.js 4.77 kB 0 B (0%)

✅ Bundle size check passed

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 83.46% 83.46% 0%
Statements 82.83% 82.83% 0%
Functions 81.40% 81.40% 0%
Branches 75.59% 75.59% 0%

Detailed Breakdown

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

✅ Coverage check passed

@github-actions

github-actions Bot commented Jul 22, 2026

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-p2k851f97-remotecom.vercel.app
Latest Commit:13630e9

Deployed with vercel-action

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-g6t1wdsuo-remotecom.vercel.app
Latest Commit:e8f0321

Deployed with vercel-action

@gabrielseco
gabrielseco merged commit c81b9a5 into main Jul 22, 2026
10 checks passed
@gabrielseco
gabrielseco deleted the add-initial-values-to-playground branch July 22, 2026 10:40
@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