Skip to content
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

getting_started.rst: add JSONOAuthLibCore as part of tutorial #734

Merged
merged 1 commit into from Aug 26, 2019
Merged

getting_started.rst: add JSONOAuthLibCore as part of tutorial #734

merged 1 commit into from Aug 26, 2019

Conversation

ghost
Copy link

@ghost ghost commented Aug 25, 2019

I had trouble getting grant_type=password working with the Django REST Framework example in the DOT tutorial. The answer was to set the OAuth2 backend class to 'oauth2_provider.oauth2_backends.JSONOAuthLibCore': #127 (comment)

I was trying to use Angular framework with its HttpClient and I think the POST requests were sending over content with application/json. It would have been a better experience to see this documented in the tutorial.

I had trouble getting `grant_type=password` working with the Django REST Framework example in the DOT tutorial. The answer was to set the OAuth2 backend class to `'oauth2_provider.oauth2_backends.JSONOAuthLibCore'`: #127 (comment)

I was trying to use Angular framework with its HttpClient and I think the POST requests were sending over content with `application/json`. It would have been a better experience to see this documented in the tutorial.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1130

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.697%

Totals Coverage Status
Change from base Build 1114: 0.0%
Covered Lines: 1250
Relevant Lines: 1320

💛 - Coveralls

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.

None yet

3 participants