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
Fix import json_available error #136
Conversation
* Flask 1.1.0 or above remove json_available. ref: d362399e Signed-off-by: chang-ning <spiderpower02@gmail.com>
Signed-off-by: chang-ning <spiderpower02@gmail.com>
By the way, Is it necessary to support Python 3.3 and old version Flask? I found Flask's compatibility with Python 3.3 is bad. |
Signed-off-by: chang-ning <spiderpower02@gmail.com>
Flask 1.1.0 dropped support for Python 3.4, so I suppose supporting it in a Flask extension doesn't make much sense. |
This needs to be merged and released to support Flask 2.0. |
Hi, I solved all conflicts. thx |
@@ -35,7 +35,7 @@ | |||
from werkzeug.utils import cached_property | |||
|
|||
# Use Flask's preferred JSON module so that our runtime behavior matches. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you move this comment down to line 51 (where the import happens)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution! Sorry that I didn't get around to this until now. I am going to submit a patch / new version to drop support for Flask < 1.0 altogether since those versions are ~8 years old now
@jcomo seems like the there hasn't been a new version since 0.8.0 in March? Would be great to have a new version that included this fix. Great repo BTW, thanks! |
@Garrett-R thanks for the ping. I just uploaded 0.8.1 to pypi. Cheers! |
Awesome, thanks! |
Hi,
Flask 1.1.0 remove json_available for compatibility. Please refer to the Flask commit.
thanks,
chang-ning