Skip to content

[Serializer] Add docs for DateTimeZoneNormalizer #12271

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 26, 2019
Merged

[Serializer] Add docs for DateTimeZoneNormalizer #12271

merged 1 commit into from
Sep 26, 2019

Conversation

wjehanne
Copy link

@wjehanne wjehanne commented Sep 5, 2019 •
edited by OskarStark
Loading

Hi,

As noticed here #12261 (comment) by @dunglas, I add the DateTimeZoneNormalizer documentation.

Fixes #12271

@wjehanne wjehanne mentioned this pull request Sep 5, 2019
Closed
@wjehanne wjehanne changed the title Add docs for DateTimeZoneNormalizer [Serializer] Add docs for DateTimeZoneNormalizer Sep 5, 2019
@OskarStark
Copy link
Contributor

This PR must target 4.3 branch, but don't worry, we can do this while merging

@OskarStark OskarStark added this to the 4.3 milestone Sep 5, 2019
@wjehanne wjehanne changed the base branch from master to 4.3 September 5, 2019 09:06
@wjehanne wjehanne changed the base branch from 4.3 to master September 5, 2019 09:07
@wjehanne wjehanne changed the base branch from master to 4.3 September 5, 2019 09:10
xabbuh
xabbuh requested changes Sep 25, 2019
View reviewed changes
javiereguiluz added a commit that referenced this pull request Sep 26, 2019
@javiereguiluz
minor #12271 [Serializer] Add docs for DateTimeZoneNormalizer (willia…
20cfcf0 
…mjehanne)

This PR was merged into the 4.3 branch.

Discussion
----------

[Serializer] Add docs for DateTimeZoneNormalizer

Hi,

As noticed here #12261 (comment) by @dunglas, I add the DateTimeZoneNormalizer documentation.

Fixes #12271

Commits
-------

320b1b2 Add docs for DateTimeZoneNormalizer
@javiereguiluz javiereguiluz merged commit 320b1b2 into symfony:4.3 Sep 26, 2019
@javiereguiluz
Copy link
Member

@williamjehanne thanks for this contribution!

@xabbuh you are right. I removed the RFC reference while merging. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xabbuh xabbuh xabbuh requested changes

@OskarStark OskarStark OskarStark approved these changes

Assignees
No one assigned
Labels
Serializer Status: Reviewed
Projects
None yet
Milestone
4.3
Development

Successfully merging this pull request may close these issues.
5 participants
@wjehanne @OskarStark @javiereguiluz @xabbuh @carsonbot