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

Fix error when clients have no associated user #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix error when clients have no associated user #92

wants to merge 1 commit into from

Conversation

rocha
Copy link

@rocha rocha commented Aug 18, 2014

The user field of the Client model is mark as nullable. However, during
serialization it is expected to be a dictionary, throwing an
error.

Fixed by checking if the user is None and serializing as null.

Review on Reviewable

The user field of the Client model is mark as nullable. However, during
serialization it is expected to be a dictionary, throwing an
error.

Fixed by checking if the user is None and serializing as null.
@clintonb
Copy link

@eculver @andy-a @devioustree @JJSaccolo @saschoar Is this project still maintained? There have been no commits for over six months, despite numerous pull requests.

@eswenson1
Copy link

@clintonb Did you ever get any response to your question of Sep 22, 2014? Should someone else take over this project? I also need this fixed and have fixed on my own fork, but this project does seem stagnant. Since django-oauth2-provider is the recommended OAuth2 framework by django-rest-framework, this really should get one or more active maintainers.

@clintonb
Copy link

@eswenson1 I never received a response.

@eswenson1
Copy link

@clintonb Thanks. It looks that there are others frustrated by lack of support and merging of PRs. I've seen references (in tickets and PRs) of others taking over the project. I'll see if I can track down anyone who is interested.

@blag
Copy link

blag commented Feb 25, 2015

@eswenson1, @clintonb: As I've mentioned elsewhere, the two most active forks are @mapmyfitness (here) and @glassresistor (here).

This upstream appears to be dead.

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.

4 participants