valohai / django-allauth-2fa

Two-factor authentication for Django Allauth
Other
212 stars 50 forks source link

Allow customizing OTP issuer and label in subclasses #157

Closed akx closed 1 year ago

akx commented 1 year ago

Fixes #53

codecov-commenter commented 1 year ago

Codecov Report

Merging #157 (5f7c36b) into main (0fc0100) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
+ Coverage   89.38%   89.42%   +0.04%     
==========================================
  Files          14       14              
  Lines         471      473       +2     
==========================================
+ Hits          421      423       +2     
  Misses         50       50              
Impacted Files Coverage Δ
allauth_2fa/utils.py 100.00% <100.00%> (ø)
allauth_2fa/views.py 83.33% <100.00%> (+0.85%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

akx commented 1 year ago

Closing in favor of #156 – it's probably better to have a single function for this.