Set JSONEncoder to json._default

wtforms / flask-wtf

Simple integration of Flask and WTForms, including CSRF, file upload and Recaptcha integration.

https://flask-wtf.readthedocs.io

BSD 3-Clause "New" or "Revised" License

1.46k stars 310 forks source link

Set JSONEncoder to json._default #536

Closed pythops closed 1 year ago

pythops commented 1 year ago

fixes #528

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in docs/changes.rst summarizing the change and linking to the issue. Add .. versionchanged:: entries in any relevant code docs.

pythops commented 1 year ago

@azmeuk can you please take a look at that one. this is blocking to use the latest version of Quart

azmeuk commented 1 year ago

It looks like JSONEncoder is not even used anymore. It was introduced with 6dea39dd2f130571c0028f1de6e489260fa01d47 and was not called anymore with 92c6e3953c3bdf2ff3cf03c1216648e6e0b77a0a. I think you can just remove the line.

pythops commented 1 year ago

Even better ! Removed.