Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Update checkbox_compound.rst.inc#3568
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
:) For the next time: You can go to your fork and the correct branch and then update the file in that branch. Then you previous PR gets updated. |
| @@ -4,5 +4,5 @@ compound | |||
| **type**: ``boolean``**default**: ``false`` | |||
| This option specifiesif a form is compound. As it's not the | |||
| case for checkbox, by fefault the value isoverriden with | |||
| case for checkbox, by fefault the value isoverridden with | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
that was fixed in#3567
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
See the other PR. Sorry I didn't combine them.
you could also use why you did not correct the other typo? fefault? |
@cordoval see the 2 comments on your typo. |
No worries Joshua - thanks for the fixes - 2 on one line! ;) |
This PR was merged into the 2.3 branch.Discussion----------Update checkbox_compound.rst.incSorry for the quick PRs in succession. Blame it on craue in #symfonyCommits-------4490dab Update checkbox_compound.rst.inc
Sorry for the quick PRs in succession. Blame it on craue in #symfony