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

Allow header OR body overwrite in response md #272

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BCVisin
Copy link
Contributor

@BCVisin BCVisin commented Jan 25, 2016

Sorta related to #3. It looks like I can overwrite the mark down template output by setting response_example in the JSON schema link. But I have to overwrite the returned headers AND body. This would allow me to simply overwrite the response headers OR body, or both.

Sorta related to interagent#3. It looks like I can overwrite the mark down template output by setting `response_example` in the JSON schema link.  But I have to overwrite the returned headers AND body.  This would allow me to simply overwrite the response headers OR body, or both.
@@ -47,7 +47,7 @@
#### Response Example

```
<%- if response_example %>
<%- if response_example.has_key?('head') %>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we might need to check if it is nil first, then do has_key? Otherwise it will blow up on has_key? undefined for nil I think.

@geemus
Copy link
Member

geemus commented Feb 23, 2016

Hey, sorry for the delay, a bit behind on open source stuff. Sounds like a reasonable request, I've added a couple comments about some of the specifics. 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.

2 participants