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

Proposed fix for #840 #841

Merged
merged 1 commit into from Sep 26, 2014

Conversation

Projects
None yet
2 participants
@ReneZwanenburg
Contributor

ReneZwanenburg commented Sep 23, 2014

No description provided.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Sep 26, 2014

Member

Thanks for the fix, that's actually a pretty nasty oversight on my part. I think I'll also deprecate the corresponding concatenation operator - better error out instead of silently accepting such a non-obvious ambiguous operation.

Member

s-ludwig commented Sep 26, 2014

Thanks for the fix, that's actually a pretty nasty oversight on my part. I think I'll also deprecate the corresponding concatenation operator - better error out instead of silently accepting such a non-obvious ambiguous operation.

s-ludwig added a commit that referenced this pull request Sep 26, 2014

Merge pull request #841 from ReneZwanenburg/fix-jsonserializer-nested…
…-arrays

Introduce `Json.appendArrayElement` to be able to append nested arrays. Fixes #840.

@s-ludwig s-ludwig merged commit b20c40b into vibe-d:master Sep 26, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment