Remove deprecated directive#18479

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

Jump to bottom

Merged

OskarStark merged 1 commit intosymfony:7.0fromOskarStark:feature/remove-deprecation

Jul 3, 2023

Merged

Remove deprecated directive#18479

OskarStark merged 1 commit intosymfony:7.0fromOskarStark:feature/remove-deprecation

Jul 3, 2023

Conversation

Copy link

Contributor

OskarStark commentedJun 30, 2023

Follows

[Form] Document a deprecation related tomodel_timezone #18475

Remove deprecated directive

e46a7a0

OskarStark requested a review fromjaviereguiluz

June 30, 2023 17:48

OskarStark requested a review fromxabbuh as acode owner

June 30, 2023 17:48

carsonbot added the Status: Needs Review label

Jun 30, 2023

carsonbot added this to the7.0 milestone

Jun 30, 2023

javiereguiluz approved these changes

Jun 30, 2023

View reviewed changes

Copy link

Member

javiereguiluz left a comment

Choose a reason for hiding this comment

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

I wondered if it'd be better to reword this instead of removing it. But, since he behavior is very obvious (you are defining the timezone of your model data here ... so your model data must have this timezone; it's so obvious). So, let's remove it. Thanks!