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

Add email to example #578

Closed pickfire closed 1 year ago

pickfire commented 1 year ago

Description

Add email to example.

Motivation and Context

Allow user to try it out easier.

How Has This Been Tested?

Manually.

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

claudep commented 1 year ago

Maybe rebase on latest master to see if errors are gone?

codecov[bot] commented 1 year ago

Codecov Report

Merging #578 (d5302de) into master (6817d28) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #578   +/-   ##
=======================================
  Coverage   95.56%   95.56%           
=======================================
  Files          74       74           
  Lines        3182     3182           
  Branches      358      358           
=======================================
  Hits         3041     3041           
  Misses        112      112           
  Partials       29       29

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