Skip to content
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

Transact literal should accept EDN and Transit as well as JSON #45

Open
mtnygard opened this issue Mar 3, 2017 · 1 comment
Open

Transact literal should accept EDN and Transit as well as JSON #45

mtnygard opened this issue Mar 3, 2017 · 1 comment
Assignees
Labels
0.9.4 Include in 0.9.4 release primed

Comments

@mtnygard
Copy link
Contributor

mtnygard commented Mar 3, 2017

Description

Expected Behavior

Vase literals should be consistent in their allowed inputs. Any of them that take data in (e.g., query and transact) should accept all the data-like representations we support:

  • JSON
  • EDN
  • Transit

Actual Behavior

The transact literal only accepts JSON payloads.

Environment

Pedestal and Vase version

Vase 0.9.1

@0xqd
Copy link
Contributor

0xqd commented Mar 6, 2017

I would love to see transit-params support since transit is much more superior than json and we already use it as standard protocol in many of our services.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.9.4 Include in 0.9.4 release primed
Projects
None yet
Development

No branches or pull requests

4 participants