urls for phone setup and delete are botched - Githubissues

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

urls for phone setup and delete are botched #655

Closed jpaniagualaconich closed 1 year ago

jpaniagualaconich commented 1 year ago

Expected Behavior

They should be:

'/account/two_factor/phone/register/'
'/account/two_factor/phone/unregister/42/'.

Current Behavior

They are:

'/account/two_factor/phone/account/two_factor/backup/phone/register/'
'/account/two_factor/phone/account/two_factor/backup/phone/unregister/42/'.

Steps to Reproduce (for bugs)

$ DJANGO_SETTINGS_MODULE=tests.settings PYTHONPATH=. django-admin shell
>>> from django.urls import reverse
>>> reverse('two_factor:phone_create')
'/account/two_factor/phone/account/two_factor/backup/phone/register/'
>>> reverse('two_factor:phone_delete', args=(42,))
'/account/two_factor/phone/account/two_factor/backup/phone/unregister/42/'