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

dependencies: fix missing email_validator [more] #312

Merged

Conversation

fenekku
Copy link
Contributor

@fenekku fenekku commented Apr 27, 2020

This is a 'quick' fix for the 2.3.1 breaking change in wtforms
that requires the email-validator library. WTForms is brought in
from flask-security and flask-security is a dependency of
invenio-accounts.

The real fix would be switching to flask-security-too since
flask-security is not maintained anymore. However, a cursory attempt
has generated more errors. So until we can address those errors,
this fix can do. (things like security.forgot_password not resolving to a route...)

  • also fixes pytest dependency issues

This is a 'quick' fix for the 2.3.1 breaking change in wtforms
that requires the email-validator library. WTForms is brought in
from flask-security and flask-security is a dependency of
invenio-accounts.

The real fix would be switching to 'flask-security-too` since
flask-security is not maintained anymore. However, a cursory attempt
has generated more errors. So until we can address those errors,
this fix can do.

- also fixes pytest dependency issues
@fenekku fenekku added this to In progress in InvenioRDM April Board via automation Apr 27, 2020
@fenekku fenekku moved this from In progress to Blocked in InvenioRDM April Board Apr 27, 2020
@lnielsen lnielsen changed the base branch from master to maint-v1.1 April 28, 2020 06:22
@lnielsen lnielsen changed the base branch from maint-v1.1 to master April 28, 2020 06:22
@lnielsen lnielsen self-assigned this Apr 28, 2020
@lnielsen lnielsen merged commit 3f8b3a7 into inveniosoftware:master Apr 28, 2020
InvenioRDM April Board automation moved this from Blocked to Done Apr 28, 2020
@fenekku fenekku deleted the fix_missing_email_validator_dep branch April 28, 2020 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants