Skip to content

Conversation

@aryaniyaps
Copy link
Contributor

Because the app_name attribute is not set in the included module (oauth2_provider), we cannot directly include endpoints with the namespace oauth2_provider.
We need to pass in a tuple of the endpoints and app-name instead. Document the same in the tutorial example.

Description of the Change

Update tutorial documentation regarding namespace inclusion

Checklist

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

Because the app_name attribute is not set in the included module (oauth2_provider), we cannot directly include endpoints with the namespace oauth2_provider.
We need to pass in a tuple of the endpoints and app-name instead. Document the same in the tutorial example.
@codecov
Copy link

codecov bot commented Mar 17, 2021

Codecov Report

Merging #945 (ab38128) into master (cc767e4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #945   +/-   ##
=======================================
  Coverage   96.59%   96.59%           
=======================================
  Files          31       31           
  Lines        1704     1704           
=======================================
  Hits         1646     1646           
  Misses         58       58           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc767e4...ab38128. Read the comment docs.

@n2ygk n2ygk added this to the 1.5.1 milestone Mar 18, 2021
Copy link
Contributor

@auvipy auvipy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@aryaniyaps
Copy link
Contributor Author

Commit?

@auvipy auvipy merged commit 592398c into django-oauth:master Mar 21, 2021
@aryaniyaps aryaniyaps deleted the patch-1 branch March 21, 2021 10:02
@n2ygk
Copy link
Contributor

n2ygk commented Mar 22, 2021

Looks like @auvipy merged this early so it's part of release 1.5.0.

@n2ygk n2ygk modified the milestones: 1.5.1, 1.5.0 Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants