jmcarp / jmcarp/flask-apispec

Default locations to `query` for GET calls

Open
#142 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Python
Stars
652
Forks
151
PR merge metrics
No merged PRs in 30d

Description

I was using:
```python
@use_kwargs({'refs': fields.Str(required=True)})
def get(self, refs):
```
And in the swagger UI the place to enter this came up as "Body". and when I tried using this, it gave me an error saying:
`TypeError: Failed to execute 'fetch' on 'Window': Request with GET/HEAD method cannot have body.`

Then I realized I had to do:
```python
@use_kwargs({'refs': fields.Str(required=True)}, locations=('query',))
def get(self, refs):
```

For get requests, it would be nice if the default value for GET requests could be set to something that doesn't error out as this was a fair bit confusing ...

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.