Skip to content
This repository was archived by the owner on Sep 13, 2024. It is now read-only.

2 small changes#10

Open
lunetics wants to merge 2 commits intoDaMitchell:masterfrom
lunetics:master
Open

2 small changes#10
lunetics wants to merge 2 commits intoDaMitchell:masterfrom
lunetics:master

Conversation

@lunetics
Copy link
Copy Markdown
Contributor

No description provided.

@lunetics
Copy link
Copy Markdown
Contributor Author

Please don't merge yet..
I propose a better idea and use a custom jms deserialization visitor, which uses iconv only if there's an json decode error.

@DaMitchell
Copy link
Copy Markdown
Owner

Hey, sounds good to me, let me know when you are ready for me to merge.

I have just seen on packagist that people are actually installing this, looks like I should actually pay more attention to this lol.

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.

2 participants