-
Notifications
You must be signed in to change notification settings - Fork 834
Fix documentation build warnings #1648
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -10,4 +10,3 @@ Django OAuth Toolkit provides a set of pre-defined views for different purposes: | |||||
| class_based | ||||||
| application | ||||||
| token | ||||||
| mixins | ||||||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sphinx complains:
This is already fully documented here: django-oauth-toolkit/docs/views/details.rst Lines 18 to 19 in dbedfaa
Since the other reference has more context and the mixins are indeed more internal than user-facing, keep only those docs. |
||||||
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -562,7 +562,8 @@ class Meta(AbstractRefreshToken.Meta): | |||
| class AbstractIDToken(models.Model): | ||||
| """ | ||||
| An IDToken instance represents the actual token to | ||||
| access user's resources, as in :openid:`2`. | ||||
| access user's resources, as in | ||||
| `OpenID Connect Core 1.0 Section 2 <https://openid.net/specs/openid-connect-core-1_0.html#IDToken/>`_. | ||||
|
Comment on lines
+565
to
+566
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Other parts of this file use a syntax like django-oauth-toolkit/docs/rfc.py Line 11 in dbedfaa
No such support actually exists for a However, it's apparent that this is the intended external link. |
||||
|
|
||||
| Fields: | ||||
|
|
||||
|
|
||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sphinx complains:
consolemakes more sense: https://pygments.org/docs/lexers/#pygments.lexers.shell.BashSessionLexer