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

Parse bool correctly #916

Merged

jowlee merged 1 commit into master from jowlee-bool-param

Oct 9, 2020

Contributor

jowlee commented Oct 9, 2020

Trello Card Link

na

Description

Fix Call to tracks endpt with show_unlisted=false don't work correctly, 400
Link to docs https://flask-restx.readthedocs.io/en/latest/parsing.html#advanced-types-handling
Link to input bool docs https://flask-restx.readthedocs.io/en/latest/api.html#flask_restx.inputs.boolean

Services

Discovery Provider

Does it touch a critical flow like Discovery indexing, Creator Node track upload, Creator Node gateway, or Creator Node file system?

Delete an option.

✅ Nope

How Has This Been Tested?

ran locally


          Parse bool correctly

17fe623

jowlee requested a review from piazzatron

October 9, 2020 20:05

piazzatron approved these changes

View reviewed changes

Contributor

piazzatron left a comment

Nice find. I'm surprised all the int ones work given that it requires this input package

jowlee merged commit 9870f49 into master

jowlee deleted the jowlee-bool-param branch

October 9, 2020 20:19

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