Skip to content

fix(tel-field) - fix tel field props#980

Merged
gabrielseco merged 3 commits intomainfrom
correct-tel-component-types
Apr 29, 2026
Merged

fix(tel-field) - fix tel field props#980
gabrielseco merged 3 commits intomainfrom
correct-tel-component-types

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented Apr 29, 2026

Note

Low Risk
Primarily a TypeScript type/prop refactor with small test harness adjustments; main risk is downstream compile/runtime issues if any consumers relied on the old TelField-local TelFieldDataProps shape.

Overview
Fixes TelField prop/type modeling by moving TelFieldDataProps out of TelField.tsx into src/types/fields.ts and basing it on FieldDataProps (rather than JSFField), so custom tel components consistently receive the same fieldData shape.

Introduces a clearer TelFieldProps type (adds required name and optional component), updates TelField and its tests to use the new types, and makes the test form resolver conditional on schema being provided.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 211.33 kB 211.4 kB -66 B (-0.0%) 🟢
Total (raw) 570.58 kB 570.58 kB 0 B (0%) 🟢
CSS (gzip) 20.69 kB 20.69 kB 0 B (0%) 🟢
CSS (raw) 107.95 kB 107.95 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 211.33 kB / 250 kB (84.5%)
  • ✅ Total raw: 570.58 kB / 600 kB (95.1%)
  • ✅ CSS gzipped: 20.69 kB / 25 kB (82.8%)

Largest Files (Top 5)

  1. chunk-3OJQ3OSP.js - 13.9 kB (0 B (0%))
  2. styles.css - 10.35 kB (0 B (0%))
  3. index.css - 10.35 kB (0 B (0%))
  4. index.js - 6.06 kB (-6 B (-0.1%))
  5. chunk-TNQKP2XT.js - 5.89 kB (0 B (0%))
View All Files (313 total)
File Size (gzip) Change
chunk-3OJQ3OSP.js 13.9 kB 0 B (0%)
styles.css 10.35 kB 0 B (0%)
index.css 10.35 kB 0 B (0%)
index.js 6.06 kB -6 B (-0.1%)
chunk-TNQKP2XT.js 5.89 kB 0 B (0%)
chunk-4SEOLHKR.js 5.81 kB new
chunk-HH474SHC.js 4.51 kB new
chunk-Y7L6VZOT.js 4.29 kB 0 B (0%)
chunk-DPVXTQAB.js 4.06 kB 0 B (0%)
chunk-N2PSMOMR.js 3.73 kB 0 B (0%)

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 89.95% 89.95% 0%
Statements 89.62% 89.62% 0%
Functions 87.69% 87.69% 0%
Branches 80.07% 80.07% 0%

Detailed Breakdown

Lines Coverage
  • Covered: 3650 / 4058
  • Coverage: 89.95%
  • Change: 0% (0 lines)
Statements Coverage
  • Covered: 3711 / 4141
  • Coverage: 89.62%
  • Change: 0% (0 statements)
Functions Coverage
  • Covered: 997 / 1137
  • Coverage: 87.69%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 2266 / 2830
  • Coverage: 80.07%
  • Change: 0% (0 branches)

✅ Coverage check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-k10mg1kdx-remotecom.vercel.app
Latest Commit:0d2cadf
Inspect:View deployment

Deployed with vercel-action

@gabrielseco gabrielseco merged commit d1edb7b into main Apr 29, 2026
10 checks passed
@gabrielseco gabrielseco deleted the correct-tel-component-types branch April 29, 2026 12:42
@gabrielseco gabrielseco mentioned this pull request May 4, 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