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

Call explicit form rendering methods to avoid warnings #555

Closed claudep closed 2 years ago

claudep commented 2 years ago

Starting from Django 4.1, default rendering of {{ form }} produces deprecation warnings. This should avoid those produced by d2fa own templates.

codecov[bot] commented 2 years ago

Codecov Report

Merging #555 (883e933) into master (a02b0ec) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #555   +/-   ##
=======================================
  Coverage   95.67%   95.67%           
=======================================
  Files          65       65           
  Lines        2938     2938           
  Branches      329      329           
=======================================
  Hits         2811     2811           
  Misses        103      103           
  Partials       24       24

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

claudep commented 2 years ago

Thanks, however you have to approve by a review to allow me to push this (that's a rule of this package).

claudep commented 2 years ago

:+1: