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.

Sign up for GitHub

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

Jump to bottom

Detect empty dynamic path params and return error #8

Closed
smollweide opened this issue Apr 6, 2016 · 0 comments
Closed

Detect empty dynamic path params and return error #8

smollweide opened this issue Apr 6, 2016 · 0 comments
Assignees
@smollweide
Labels
enhancement
Milestone
0.7.2

Comments

@smollweide
Copy link
Owner

As suggested by JonWallsten

Also i don't figure out how to show error.json if no ID is supplied when doing the GET.
@smollweide smollweide added this to the 0.7.2 milestone Apr 6, 2016
@smollweide smollweide self-assigned this Apr 6, 2016
@smollweide smollweide mentioned this issue Apr 6, 2016
Closed
smollweide pushed a commit that referenced this issue Apr 7, 2016
[change] #8: add hasEmptyDynamicPathParam method
6e5b355
smollweide pushed a commit that referenced this issue Apr 7, 2016
[test] #8: update tests for getResponse
dd6de8f
smollweide pushed a commit that referenced this issue Apr 7, 2016
[test] #8: add new example call
087c396
smollweide pushed a commit that referenced this issue Apr 7, 2016
[bugfix] #8: change GetResponse - replace dynamic path params
f09a766
smollweide pushed a commit that referenced this issue Apr 7, 2016
[test] #8: add test for empty dynamic path
b416730
smollweide pushed a commit that referenced this issue Apr 7, 2016
[test] #8: add test for placeholder dynamic path
192fe8f
smollweide pushed a commit that referenced this issue Apr 7, 2016
[change] #8: rename hasEmptyDynamicPathParam method to hasValidDynami…
9f94e53 
…cPathParam
smollweide pushed a commit that referenced this issue Apr 7, 2016
[change] #8: rename call /search/products/{productCode}/available to …
3233631 
…/search/users/{userId}/products/{productCode}/available
smollweide pushed a commit that referenced this issue Apr 7, 2016
[test] #8: add test for multiple dynamic path params invalid
c0bd399
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smollweide smollweide

Labels
enhancement
Projects
None yet
Milestone
0.7.2
Development

No branches or pull requests
1 participant
@smollweide