Skip to content

fix(radio): ngModelChange fires multiple times #9164

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
Merged
kdinev merged 2 commits into from
Mar 17, 2021
Merged

fix(radio): ngModelChange fires multiple times #9164

kdinev merged 2 commits into from
Mar 17, 2021

Conversation

@simeonoff
Copy link
Collaborator

@simeonoff simeonoff commented Mar 16, 2021

Closes #9155

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@simeonoff simeonoff requested review from kdinev and wnvko March 16, 2021 16:04
@simeonoff simeonoff linked an issue Mar 16, 2021 that may be closed by this pull request
IgxRadio fires ngModelChange when same option is selected #9155
Closed
@wnvko wnvko added ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification labels Mar 17, 2021
@wnvko wnvko removed their assignment Mar 17, 2021
@kdinev kdinev merged commit 57178c5 into 10.2.x Mar 17, 2021
@kdinev kdinev deleted the simeonoff/fix-9155-10.2.x branch March 17, 2021 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wnvko wnvko wnvko approved these changes

@kdinev kdinev Awaiting requested review from kdinev

Assignees

No one assigned

Labels

radio-button version: 10.2.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

IgxRadio fires ngModelChange when same option is selected

4 participants

@simeonoff @wnvko @kdinev