yada 2019-02-06

@nmdrenard has left the channel

@grierson has joined the channel

Is it possible that Yada doesn't support metadata with transit?

Browsing the source I don't see {:transform t/write-meta} like in the transit tests. https://github.com/cognitect/transit-clj/blob/master/test/transit/test_with_meta.clj

It's possible. The transit code was contributed quite a long time ago. I didn't actually know of the metadata feature. Feel free to fork the extension.

Thanks, I'll give it a look.

Yup, tested and it works by adding {:transform transit/write-meta}. 🙂

👍 1

Send a PR!

Really? Isn't it a breaking behavior?

why is it breaking? 🤔

Currently the metadata is stripped. With this change the server automatically sends the metadata, even if it's a huge map.

old programs will continue to work though I suppose.

➕ 1