jazzband / django-two-factor-auth

Complete Two-Factor Authentication for Django providing the easiest integration into most Django projects.
MIT License
1.71k stars 448 forks source link

added missing extra_media block in example base template #551

Closed jpaniagualaconich closed 1 year ago

jpaniagualaconich commented 2 years ago

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist:

jpaniagualaconich commented 2 years ago

only now i realise example has its own base template 😅️

codecov[bot] commented 2 years ago

Codecov Report

Merging #551 (7faead3) into master (c086b7f) will increase coverage by 10.52%. The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #551       +/-   ##
===========================================
+ Coverage   87.63%   98.15%   +10.52%     
===========================================
  Files          65       65               
  Lines        2862     2932       +70     
  Branches      261      290       +29     
===========================================
+ Hits         2508     2878      +370     
+ Misses        337       30      -307     
- Partials       17       24        +7     
Impacted Files Coverage Δ
tests/test_admin.py 100.00% <0.00%> (ø)
tests/test_email.py 100.00% <0.00%> (ø)
tests/test_utils.py 100.00% <0.00%> (ø)
tests/test_yubikey.py 100.00% <0.00%> (ø)
tests/test_gateways.py 100.00% <0.00%> (ø)
tests/test_views_login.py 100.00% <0.00%> (ø)
tests/test_views_phone.py 100.00% <0.00%> (ø)
tests/test_views_setup.py 100.00% <0.00%> (ø)
tests/test_views_mixins.py 100.00% <0.00%> (ø)
tests/test_views_qrcode.py 100.00% <0.00%> (ø)
... and 18 more

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

jpaniagualaconich commented 1 year ago

@claudep @moggers87 there's also this one that I overlooked.

jpaniagualaconich commented 1 year ago

thanks! 🎉