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

Document parameter defaults #59

Open
ddeaguiar opened this issue Mar 23, 2017 · 3 comments
Open

Document parameter defaults #59

ddeaguiar opened this issue Mar 23, 2017 · 3 comments

Comments

@ddeaguiar
Copy link
Contributor

Description

Let's make this more visible in the docs.

@ddeaguiar
Copy link
Contributor Author

A recent commit adds a note regarding default parameters in the Handling Parameters section of the Your First API guide but I still think we should include this documentation elsewhere as well. Perhaps the Action Literals document?

@ohpauleez
Copy link
Contributor

Default parameters were added in the lastest evolution of Vase, which is why the documentation is light. I agree, we should mention it in the docs (the samples and tests illustrate it)

@mtnygard
Copy link
Contributor

I think we should add a new doc that is all about parameters. There is more subtlety that we should cover:

  • Dealing with namespaced names from JSON
  • The order of merging params. Who wins if there is a query-param, a path-param, and a body-param with the same key?
  • The role of edn-coerce.

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

No branches or pull requests

3 participants