Skip to content

update on docs and comments for default option value for checkbox #21230

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

Merged
merged 2 commits into from
Dec 8, 2020
Merged

update on docs and comments for default option value for checkbox #21230

merged 2 commits into from
Dec 8, 2020

Conversation

adhispace
Copy link
Contributor

updates the documentation and comments of customizing the default option of the checkbox.
the default option value to provide for checkbox is MAT_CHECKBOX_DEFAULT_OPTIONS for version>=10

fixes #21025

adhispace and others added 2 commits December 6, 2020 19:45
… value for checkbox

updates the documentation and comments of customizing the default option of the checkbox.
the default option value to provide for checkbox is MAT_CHECKBOX_DEFAULT_OPTIONS for version>=10

fixes #21025
docs(material/checkbox): updated docs and comments for default option…
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 6, 2020
@adhispace
Copy link
Contributor Author

Hi

this PR has changes as same as the below one
#21226

I closed the PR #21226 as I faced difficulty in resolving circleci build error as this is my first ever PR. sorry for the trouble.
kindly review.

thanks

Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@crisbeto crisbeto added merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note docs This issue is related to documentation action: merge The PR is ready for merge by the caretaker merge safe target: patch This PR is targeted for the next patch release labels Dec 6, 2020
@crisbeto
Copy link
Member

crisbeto commented Dec 6, 2020

Caretaker note: for some reason the CI didn't run on this PR. It's only doc changes so I think that we can merge it anyway.

Copy link
Contributor

@wagnermaciel wagnermaciel left a comment

Choose a reason for hiding this comment

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

lgtm

@jelbourn
Copy link
Member

jelbourn commented Dec 7, 2020

@adhispace do you happen to have CircleCI set up on your fork of angular/components?

@wagnermaciel
Copy link
Contributor

Seems like this change has conflicts with 11.0.x. Is it ok to change this to target: minor?

@adhispace
Copy link
Contributor Author

@adhispace do you happen to have CircleCI set up on your fork of angular/components?

no, I dont have CircleCI setup in my fork.

@wagnermaciel wagnermaciel added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Dec 8, 2020
@wagnermaciel wagnermaciel merged commit a8bb1f3 into angular:master Dec 8, 2020
wagnermaciel pushed a commit that referenced this pull request Dec 8, 2020
… value for checkbox (#21230)

updates the documentation and comments of customizing the default option of the checkbox.
the default option value to provide for checkbox is MAT_CHECKBOX_DEFAULT_OPTIONS for version>=10

fixes #21025
wagnermaciel added a commit that referenced this pull request Dec 8, 2020
… value for checkbox (#21230) (#21254)

updates the documentation and comments of customizing the default option of the checkbox.
the default option value to provide for checkbox is MAT_CHECKBOX_DEFAULT_OPTIONS for version>=10

fixes #21025

Co-authored-by: adhispace <[email protected]>
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jan 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement docs This issue is related to documentation merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: minor This PR is targeted for the next minor release
Projects
None yet
4 participants