Simplify urlpattern loading

jazzband / django-oauth-toolkit

OAuth2 goodies for the Djangonauts!

https://django-oauth-toolkit.readthedocs.io

Other

3.06k stars 777 forks source link

Simplify urlpattern loading #1436

Closed jaap3 closed 1 week ago

jaap3 commented 1 week ago

Description of the Change

Simplify how urlpatterns are loaded, as requested in https://github.com/jazzband/django-oauth-toolkit/pull/1435#pullrequestreview-2131162185

Checklist

[x] PR only contains one change (considered splitting up PR)
[ ] unit-test added
[x] documentation updated
[ ] CHANGELOG.md updated (only for user relevant changes)
[x] author name in AUTHORS

jaap3 commented 1 week ago

@n2ygk as requested, here's a PR that changes how the urlpatterns are loaded. I've also updated how they are loaded in tests, to verify that it works.

This branch is based on PR https://github.com/jazzband/django-oauth-toolkit/pull/1435.

I didn't want to update that PR, because this change is somewhat more involved than just a minor docs tweak.

I'll rebase this one when the base branch merges.

[EDIT]: Not sure why the readthedocs build has failed? Doesn't seem related to the changes.