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.

Sign up for GitHub

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

Jump to bottom

Capitalized Yes and No defaults #1395

Merged
merged 5 commits into from Dec 7, 2019
Merged

Capitalized Yes and No defaults #1395

merged 5 commits into from Dec 7, 2019

Conversation

markbmullins
Copy link
Contributor

@markbmullins markbmullins commented Aug 10, 2019 •
edited by epicfaace

Reasons for making this change

This change was requested in issue #1315 Capitalize "yes"/"no" by default for the default radio widget. I have updated the BooleanFields component as well as the associated tests.

Fixes #1315

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests
    • I've updated docs if needed
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

epicfaace
epicfaace approved these changes Dec 7, 2019
View reviewed changes
Copy link
Member

@epicfaace epicfaace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@epicfaace epicfaace merged commit 4f02482 into rjsf-team:master Dec 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@epicfaace epicfaace epicfaace approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Capitalize "yes"/"no" by default for the default radio widget
2 participants
@markbmullins @epicfaace