Skip to content

[Serializer] Use pack to install #11716

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
Jun 11, 2019
Merged

[Serializer] Use pack to install #11716

merged 1 commit into from
Jun 11, 2019

Conversation

TimoBakx
Copy link
Member

@TimoBakx TimoBakx commented Jun 10, 2019

As here, I also ran into issues using messenger with the Symfony Serializer, due to symfony/property-access missing and the ObjectNormalizer not being available due to that.

This change updates the docs to use the pack instead of the package to install the Serializer (which includes symfony/property-access).

I also included a note for developers that already installed the Serializer directly and try to figure out why ObjectNormalizer wasn't available.

@OskarStark OskarStark added this to the 4.2 milestone Jun 11, 2019
@OskarStark
Copy link
Contributor

Thank you Timo.

@OskarStark OskarStark merged commit 1ffc677 into symfony:4.2 Jun 11, 2019
OskarStark added a commit that referenced this pull request Jun 11, 2019
This PR was merged into the 4.2 branch.

Discussion
----------

[Serializer] Use pack to install

As [here](symfony/symfony#31490 (comment)), I also ran into issues using messenger with the Symfony Serializer, due to `symfony/property-access` missing and the `ObjectNormalizer` not being available due to that.

This change updates the docs to use the pack instead of the package to install the Serializer (which includes `symfony/property-access`).

~I also included a note for developers that already installed the Serializer directly and try to figure out why `ObjectNormalizer` wasn't available.~

Commits
-------

1ffc677 [Serializer] Use pack to install
@TimoBakx TimoBakx deleted the serializer-pack-4.2 branch October 28, 2019 08:21
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.

4 participants