Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed setting initial value for enum type #1354

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

piotrpospiech
Copy link
Collaborator

  • Removed setting initial value for enum type (SelectField). This is the same behavior as we have for example in the SimpleSchema2Bridge

Before

image

After

image

Future work

@piotrpospiech piotrpospiech added this to the v4.0 milestone Jun 18, 2024
@github-actions github-actions bot added Area: Bridge Affects some of the bridge packages Bridge: Zod Affects the uniforms-bridge-zod package labels Jun 18, 2024
@github-actions github-actions bot added Area: Core Affects the uniforms package Area: Theme Affects some of the theme packages Theme: MUI Affects the uniforms-mui package labels Oct 23, 2024
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.53%. Comparing base (36eb052) to head (c47214a).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1354      +/-   ##
==========================================
+ Coverage   95.46%   95.53%   +0.06%     
==========================================
  Files         175      175              
  Lines        2869     2867       -2     
  Branches      775      774       -1     
==========================================
  Hits         2739     2739              
  Misses         59       59              
+ Partials       71       69       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@piotrpospiech piotrpospiech marked this pull request as ready for review October 23, 2024 15:35
@piotrpospiech piotrpospiech merged commit 38c3acf into master Oct 29, 2024
6 checks passed
@piotrpospiech piotrpospiech deleted the zod-enum-initial-value branch October 29, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Bridge Affects some of the bridge packages Area: Core Affects the uniforms package Area: Theme Affects some of the theme packages Bridge: Zod Affects the uniforms-bridge-zod package Theme: MUI Affects the uniforms-mui package
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants