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.

Sign up for GitHub

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

Jump to bottom

get_all_permissions is not available on all Authentication Backends #155

Open

arjun2504 wants to merge 2 commits into javrasya:master from arjun2504:master

arjun2504 commented May 17, 2020

For example, GraphQL JWT does not have get_all_permissions. Thus, there should be a check to identify whether the definition is supported or not.

arjun2504 added 2 commits

May 17, 2020 16:49


          Update orm_driver.py

fd1a66c


          Merge pull request #1 from arjun2504/permission_attr_check

get_all_permissions() cannot be expected on all authentication backends

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

5 similar comments

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

coveralls commented May 17, 2020

Coverage increased (+0.02%) to 80.938% when pulling 8202905 on arjun2504:master into f58df9f on javrasya:master.

Owner

javrasya commented May 18, 2020

Thank you for the PR and It makes sense @arjun2504 but let me have a look at this soon as possible and get back to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet