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.

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

1.0 version deprecations removal #484

Merged
merged 2 commits into from
Nov 7, 2021
Merged

1.0 version deprecations removal #484

merged 2 commits into from
Nov 7, 2021

Conversation

azmeuk
Copy link
Member

@azmeuk azmeuk commented Nov 7, 2021

html5 module is incompatible with wtforms 3, and its deprecation warning announce it will be remove for flask_wtf 1.0. So I suppose 1.0 is the next flask-wtf version in order to support wtforms 3.

@davidism what do you think?

Also, I do not have accesses to flask-wtf pypi to I cannot make the release myself at the moment.

@azmeuk azmeuk requested a review from davidism November 7, 2021 13:48
@davidism
Copy link
Member

davidism commented Nov 7, 2021

This library is definitely ready for 1.0. Although it would probably be a good idea to check if recaptcha still works, it seems like many of the issues are about that.

@azmeuk azmeuk merged commit 4b90067 into pallets-eco:main Nov 7, 2021
@azmeuk
Copy link
Member Author

azmeuk commented Nov 7, 2021

recaptcha v2 still works correctly

"checked during processing instead."
)
)
return bool(self.data)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@azmeuk / @davidism So how does a user check whether the file field has data? The deprecation is a triple negative and I find it hard to understand 'data' is no longer set if the input is not a non-empty". That triple negative is hard to understand.

The bit in the documentation I can find from the change log https://flask-wtf.readthedocs.io/en/1.0.x/changes/#version-0-14-1 says:

"has_file is deprecated; it’s now equivalent to bool(field.data)."

As that's the only mention in the change log I can see, I presume I can write bool(field.data) still? Which sort of makes me wonder why remove the has_file() method?

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants