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

Password scheme update at login #1579

Merged
merged 6 commits into from Oct 9, 2018

Conversation

Projects
None yet
2 participants
@Toniob
Contributor

Toniob commented Oct 5, 2018

close #1565

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Oct 5, 2018

Codecov Report

Merging #1579 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1579      +/-   ##
==========================================
+ Coverage      86%   86.04%   +0.03%     
==========================================
  Files         143      143              
  Lines        7455     7467      +12     
==========================================
+ Hits         6412     6425      +13     
+ Misses       1043     1042       -1

codecov bot commented Oct 5, 2018

Codecov Report

Merging #1579 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1579      +/-   ##
==========================================
+ Coverage      86%   86.04%   +0.03%     
==========================================
  Files         143      143              
  Lines        7455     7467      +12     
==========================================
+ Hits         6412     6425      +13     
+ Misses       1043     1042       -1

Toniob added some commits Oct 5, 2018

@modoboa modoboa deleted a comment from codacy-bot Oct 6, 2018

@modoboa modoboa deleted a comment from codacy-bot Oct 6, 2018

@modoboa modoboa deleted a comment from codacy-bot Oct 6, 2018

@modoboa modoboa deleted a comment from codacy-bot Oct 6, 2018

@modoboa modoboa deleted a comment from codacy-bot Oct 6, 2018

Show outdated Hide outdated modoboa/core/app_settings.py Outdated
@@ -12,6 +12,7 @@
from django.urls import reverse
from modoboa.lib.tests import NO_SMTP, ModoTestCase
from modoboa.core.password_hashers import get_password_hasher

This comment has been minimized.

@tonioo

tonioo Oct 7, 2018

Member

Please respect grouping and alphabetical order for import statements.

@tonioo

tonioo Oct 7, 2018

Member

Please respect grouping and alphabetical order for import statements.

@tonioo tonioo merged commit 1301900 into modoboa:master Oct 9, 2018

4 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
codecov/patch Coverage not affected when comparing cc26b4b...b88f5d9
Details
codecov/project 86.04% (+0.03%) compared to cc26b4b
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment