Skip to content

[Serializer] Remove duplicate normalizers docs #14335

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
Dec 27, 2021

Conversation

HypeMC
Copy link
Member

@HypeMC HypeMC commented Oct 4, 2020

Currently normalizers are documented in three separate places, each of which contains some information the other two don't. This makes browsing the docs difficult and confusing.

With this PR I've consolidated everything to components/serializer.rst. I've removed the list from serializer/normalizers.rst and added a link to components/serializer.rst instead.
Since not all normalizers are enabled by the Framework Bundle by default, serializer.rst still contains a list of those that are, but the descriptions have been removed and additional links to components/serializer.rst added.

@HypeMC HypeMC changed the title Remove duplicate normalizers docs [Serializer] Remove duplicate normalizers docs Oct 4, 2020
@javiereguiluz javiereguiluz added this to the 4.4 milestone Jan 8, 2021
@HypeMC HypeMC force-pushed the remove-duplicate-normalizers-docs branch from af29c1e to 21b89be Compare January 9, 2021 15:16
@HypeMC
Copy link
Member Author

HypeMC commented Jan 9, 2021

@javiereguiluz I've rebased the branch and resolved the conflict.

@HypeMC HypeMC force-pushed the remove-duplicate-normalizers-docs branch from 21b89be to d76fe1d Compare August 20, 2021 01:41
@javiereguiluz
Copy link
Member

@HypeMC, thanks a lot for this contribution. I'm sorry it took us so long to merge it

@javiereguiluz javiereguiluz merged commit d077bce into symfony:4.4 Dec 27, 2021
@HypeMC HypeMC deleted the remove-duplicate-normalizers-docs branch December 27, 2021 15:46
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.

3 participants