8000 [Serializer][RFC] Provide DateTimeZoneNormalizer · Issue #30145 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Serializer][RFC] Provide DateTimeZoneNormalizer #30145

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
kiler129 opened this issue Feb 11, 2019 · 1 comment
Closed

[Serializer][RFC] Provide DateTimeZoneNormalizer #30145

kiler129 opened this issue Feb 11, 2019 · 1 comment
Labels
RFC RFC = Request For Comments (proposals about features that you want to be discussed) Serializer

Comments

@kiler129
Copy link
Contributor
kiler129 commented Feb 11, 2019

Description
The Symfony Serializer contains support for DateTimeInterface objects which some projects rely on (e.g. API Platform - api-platform/core#2509). However object being very close to the DateTime - DateTimeZone does not provide any serialization options out of the box.

RFC
If there was a reason to provide normalization for DateTime why not include DateTimeZone which is also a part of the DateTime?

@chalasr chalasr added Serializer RFC RFC = Request For Comments (proposals about features that you want to be discussed) labels Feb 12, 2019
@jewome62
Copy link
Contributor
jewome62 commented Apr 6, 2019

I take this #EUFOSSA

jewome62 added a commit to jewome62/symfony that referenced this issue Apr 6, 2019
jewome62 added a commit to jewome62/symfony that referenced this issue Apr 6, 2019
jewome62 added a commit to jewome62/symfony that referenced this issue Apr 6, 2019
@fabpot fabpot closed this as completed Apr 6, 2019
fabpot added a commit that referenced this issue Apr 6, 2019
This PR was squashed before being merged into the 4.3-dev branch (closes #30915).

Discussion
----------

[Serializer] Add datetimezone normalizer

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #30145
| License       | MIT
| Doc PR        | not yet

Commits
-------

1546c0d [Serializer] Add datetimezone normalizer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC RFC = Request For Comments (proposals about features that you want to be discussed) Serializer
Projects
None yet
Development

No branches or pull requests

5 participants
0