8000 Update reset.rst by RobertErobert · Pull Request #7770 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Update reset.rst #7770

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

Closed
wants to merge 2 commits into from
Closed

Update reset.rst #7770

wants to merge 2 commits into from

Conversation

RobertErobert
Copy link
@RobertErobert RobertErobert commented Apr 8, 2017

Added a note that the examples comes from ButtonType::class. #6864

8000
fixed typo
Forgot the dot at the end of the sentence :)
@HeahDude
Copy link
Contributor
HeahDude commented Apr 8, 2017

Hello @DJWindless I think it will be easier to review and merge if you do all the changes in one PR, either this one or #7772.

@RobertErobert
Copy link
Author

Hello @HeahDude, sorry for this. This is my first time with github and PR. Next time i will do better :)

@HeahDude
Copy link
Contributor
HeahDude commented Apr 8, 2017

Please don't be sorry :), thank you very much for improving the docs on your first contribution!

@javiereguiluz
Copy link
Member

I'm closing this as fixed by the alternative solution provided by #7824.

@DJWindless even if your pull request wasn't merged, it helped us notice this issue. So thanks a lot for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0