Exclude serialization related to AdminApiBundle from Sylius

Sylius / SyliusAdminApiBundle

Administration REST API for Sylius eCommerce.

MIT License

11 stars 22 forks source link

Exclude serialization related to AdminApiBundle from Sylius #9

Closed AdamKasp closed 3 years ago

AdamKasp commented 3 years ago

based on #8

after we excluded AdminApiBundle from Sylius, we created a BC break with serialization, after this PR we should remove these serializations from Sylius/Sylius

and I have one doubt should we move all these serializations from Sylius/Sylius to AdminApiBundle?

Zales0123 commented 3 years ago

We should not move serializations, as they still could be used by people not using AdminApiBundle - it would be a big BC break 🖖

Zales0123 commented 3 years ago

Thank you, Adam! :tada: