Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[FrameworkBundle] Remove doctrine/annotations integration#51050
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
[FrameworkBundle] Remove doctrine/annotations integration#51050
Uh oh!
There was an error while loading.Please reload this page.
Conversation
derrabus commentedJul 20, 2023
| Q | A |
|---|---|
| Branch? | 7.0 |
| Bug fix? | no |
| New feature? | yes |
| Deprecations? | no |
| Tickets | Follows#50888 |
| License | MIT |
| Doc PR | TODO, seesymfony/symfony-docs#18589 |
44b8b76 tocbf8a2eCompare
nicolas-grekas left a comment
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.
please mind fabbot :)
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
cbf8a2e to713ff44Compare| $this->render(..., ['form' => $form]); | ||
| ``` | ||
| * Remove the integration of Doctrine annotations, use native attributes instead |
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.
This was not supposed to be removed in Symfony 8?
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.
Nope.
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.
In#50888, you said:
The plan is to make this the default and only valid option in Symfony 7 and remove that setting entirely in Symfony 8.
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.
Ohh sorry, just got it. The option cannot be enabled on Symfony 7
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.
Yes, the setting is still there. You just cannot enable it.
nicolas-grekas commentedJul 24, 2023
Thank you@derrabus. |