Conversation
|
In the patch: the title 'The Context parameter of XmlEncoder' will probably need to be entirely underlined with the correct markup (~~~~~~~~~~~~~~~~~~), so the markup test will pass. |
|
Thanks for your contribution, however I am unable to see the real difference, as most of the content is just moved around. Can you please point me to the lines which makes sth more clear than before? Thanks |
|
Sure! |
The following issue has been addressed: Repeated entry in the Serializer components doc symfony#11786 The separate documentations for XMLEncoder were put into one section with minor changes in text. The documentation now contains one single section for XMLEncoder.
|
Hi @armin-github! I'm very sorry for the long wait, I'm not sure why we've missed this. The PR makes a lot of sense, the duplicate sections should indeed be merged. |
…a table (wouterj) This PR was merged into the 4.4 branch. Discussion ---------- [Serializer] Merge encoder sections and show options in a table While merging #13269 , I found that we also documented CsvEncoder options in a different section. This merges both duplicated sections. I've also transformed them to a table, similar to the recent changes to documenting the provider options in the Messenger guide. Commits ------- d397c38 Merge encoder sections and show options in a table
4 participants
The following issue has been addressed: Repeated entry in the Serializer components doc #11786
The separate documentations for XMLEncoder were put into one section with minor changes in text. The documentation now contains one single section for XMLEncoder.