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

Skip to content

[FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler #7633

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

Merged
merged 1 commit into from
Apr 15, 2017

Conversation

Nek-
Copy link
Contributor

@Nek- Nek- commented Mar 15, 2017

This PR provides documentation for this other PR symfony/symfony#22011

It should not be merged before the related Symfony PR is merged.

It also fixes missing options in the example YAML config file.

This is added by symfony/symfony#22011
This commit also fix missing serializer related missing options in the yaml
config example.
@Nek- Nek- changed the title Add new framework bundle option to reference: circular_reference_handler [Serializer] Add new framework bundle option to reference: circular_reference_handler Mar 15, 2017
@Nek- Nek- changed the title [Serializer] Add new framework bundle option to reference: circular_reference_handler [FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler Mar 15, 2017
@javiereguiluz javiereguiluz changed the title [FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler [WCM][FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler Mar 16, 2017
@lyrixx
Copy link
Member

lyrixx commented Mar 17, 2017

LGTM 👍 ; thanks to take care of this one ;)

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

👍

@Nek-
Copy link
Contributor Author

Nek- commented Apr 11, 2017

@javiereguiluz this PR is good to merge I guess. (before it becomes outdated :p)

@xabbuh xabbuh changed the title [WCM][FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler [FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler Apr 14, 2017
@HeahDude
Copy link
Contributor

This closes #7699.

@HeahDude HeahDude added this to the 3.3 milestone Apr 15, 2017
@xabbuh
Copy link
Member

xabbuh commented Apr 15, 2017

Thank you @Nek-.

@xabbuh xabbuh merged commit 964c31e into symfony:master Apr 15, 2017
xabbuh added a commit that referenced this pull request Apr 15, 2017
…option to reference: circular_reference_handler (Nek-)

This PR was merged into the master branch.

Discussion
----------

[FrameworkBundle][Serializer] Add new framework bundle option to reference: circular_reference_handler

This PR provides documentation for this other PR symfony/symfony#22011

It should not be merged before the related Symfony PR is merged.

It also fixes missing options in the example YAML config file.

Commits
-------

964c31e Add new framework bundle option to reference: circular_reference_handler
xabbuh added a commit that referenced this pull request Apr 15, 2017
@Nek- Nek- deleted the serializer-circular-ref-handler branch April 18, 2017 14:54
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.

6 participants