Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

v3.0.0 #25

Draft
wants to merge 27 commits into
base: master
Choose a base branch
from
Draft

v3.0.0 #25

wants to merge 27 commits into from

Conversation

NicolasGuilloux
Copy link
Owner

@NicolasGuilloux NicolasGuilloux commented May 20, 2020

Breaking Change:

  • The DTO annotation has been removed.
  • You now need to implement the DataTransferObjectInterface to your DTO
  • DtoUtility::loadArrayToDto is now available through the Symfony Serializer using the denormalizer.
  • DtoUtility::updateEntityList is now available at EntityUtils::updateCollection
  • You no longer need to declare the DTO as a service

New feature:

  • Implementation of a DTO normalizer/denormalizer for Symfony's Serializer

Fixes

@NicolasGuilloux NicolasGuilloux marked this pull request as draft May 20, 2020 12:20
@NicolasGuilloux
Copy link
Owner Author

NicolasGuilloux commented May 20, 2020

Todo:

  • Check the MapTo annotation and the Field annotation from the EntityIdNormalizer to return the appropriate value.
  • A lot of tests for all Serializers classes
  • Clean the ParamConverter by being more SOLID and split the functions in different classes
  • Should fix the Invalid given value for array (500) #19 issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants