Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[FrameworkBundle] Added doc for ControllerTrait::isFormValid #8518

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 1 commit into from

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Oct 16, 2017

@javiereguiluz
Copy link
Member

Closing because the related proposal (symfony/symfony#24576) was closed without merging.

@lyrixx lyrixx deleted the form-is-valid branch February 9, 2018 15:21
@lyrixx
Copy link
Member Author

lyrixx commented Feb 9, 2018

Thanks for closing this PR. I forgot

symfony-splitter pushed a commit to symfony/framework-bundle that referenced this pull request Jan 2, 2019
… (lyrixx)

This PR was merged into the 4.3-dev branch.

Discussion
----------

[FrameworkBundle] Added `ControllerTrait::isFormValid`

| Q             | A
| ------------- | ---
| Branch?       | master (4.1)
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        | symfony/symfony-docs#8518

Commits
-------

1a8c844f0e [FrameworkBundle] Added `ControllerTrait::isFormValid`
fabpot added a commit to symfony/symfony that referenced this pull request Jan 2, 2019
… (lyrixx)

This PR was merged into the 4.3-dev branch.

Discussion
----------

[FrameworkBundle] Added `ControllerTrait::isFormValid`

| Q             | A
| ------------- | ---
| Branch?       | master (4.1)
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        | symfony/symfony-docs#8518

Commits
-------

1a8c844 [FrameworkBundle] Added `ControllerTrait::isFormValid`
@eXtreme
Copy link
Contributor

eXtreme commented Jan 6, 2019

@javiereguiluz @lyrixx shouldn't this be reopened now as symfony/symfony#24576 was merged?

@HeahDude HeahDude added this to the 4.3 milestone Jan 7, 2019
@HeahDude HeahDude removed the Waiting Code Merge Docs for features pending to be merged label Jan 7, 2019
@lyrixx lyrixx restored the form-is-valid branch January 7, 2019 09:17
@lyrixx
Copy link
Member Author

lyrixx commented Jan 7, 2019

@eXtreme You are right. I'm not able to re-open the PR. I will open a new one instead.
Thanks for the reminder

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.

5 participants