Skip to content

set radio group input value to option.value#539

Merged
chawes13 merged 4 commits intomainfrom
radio-group-input-value
Jun 14, 2022
Merged

set radio group input value to option.value#539
chawes13 merged 4 commits intomainfrom
radio-group-input-value

Conversation

@jhp0621
Copy link
Copy Markdown
Contributor

@jhp0621 jhp0621 commented Mar 19, 2022

#538

Author Checklist

  • Add unit test(s)
  • Update version in package.json (see the versioning guidelines)
  • Update documentation (if necessary)
  • Add story to storybook (if necessary)
  • Assign dev reviewer

@jhp0621 jhp0621 requested a review from chawes13 March 19, 2022 00:19
@jhp0621
Copy link
Copy Markdown
Contributor Author

jhp0621 commented Mar 19, 2022

@chawes13 not sure how to solve the version check fail.. 😬

@chawes13
Copy link
Copy Markdown
Contributor

@jhp0621 update the version in package.json 😀. Can you add a test for this please?

@jhp0621 jhp0621 changed the title pass in options.value to input's value prop set radio group input value to option.value Mar 19, 2022
Copy link
Copy Markdown
Contributor

@chawes13 chawes13 left a comment

Choose a reason for hiding this comment

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

🎉

@chawes13 chawes13 merged commit 7ba4ac8 into main Jun 14, 2022
@chawes13 chawes13 deleted the radio-group-input-value branch June 14, 2022 13:54
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