Skip to content

Conversation

@temp
Copy link

@temp temp commented Aug 19, 2016

It didn't work with xml here - json worked fine.
Code adapted to JMS\Serializer\Handler\DateHandler::serializeDateTime().
Works fine now.

@coveralls
Copy link

coveralls commented Aug 19, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 446b8b1 on temp:fix-serialize-uuid into 6d9d3b5 on mhujer:master.

@mhujer
Copy link
Owner

mhujer commented Aug 22, 2016

Thanks! 👍

I've merged it manually (24e419c) and added tests (a94423e).

Can you please verify that the master branch works fine for you? (I'll tag it afterwards).

@temp
Copy link
Author

temp commented Aug 22, 2016

Just tested the master locally, looks good! 👍

@mhujer
Copy link
Owner

mhujer commented Aug 22, 2016

That was quick! :-)

I've released it as 1.0.1

@mhujer mhujer closed this Aug 22, 2016
@temp
Copy link
Author

temp commented Aug 22, 2016

Great, thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants