Skip to content
This repository was archived by the owner on Apr 12, 2023. It is now read-only.

Set can_issue_permission to organizations by default - #56

Open
Khukhunashvili wants to merge 1 commit into
JBossOutreachArchive:masterfrom
Khukhunashvili:permission
Open

Set can_issue_permission to organizations by default#56
Khukhunashvili wants to merge 1 commit into
JBossOutreachArchive:masterfrom
Khukhunashvili:permission

Conversation

@Khukhunashvili

@Khukhunashvili Khukhunashvili commented Dec 4, 2018

Copy link
Copy Markdown
Contributor

Currently, I'm working on GCI task where I have to issue a certificates, but when I log in as organization & send post request to /api/issue_certificate, it responds with message:
You do not have permission to perform this action., so this (Giving 'can_issue_certificate' permission' to organization by default) should fix the problem.

please let me know if I'm missing something & there is a way to get this permission.

@aashutoshrathi aashutoshrathi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah, we needed it bad.
Thanks @khukhuna

@jatinkatyal13 jatinkatyal13 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice work! But can't we handle this in the model rather than serializers?

@aashutoshrathi

Copy link
Copy Markdown
Collaborator

@khukhuna We might need a rebase buddy 😅

Comment thread api/serializers.py Outdated
Comment thread api/serializers.py Outdated
Comment thread api/serializers.py

@aashutoshrathi aashutoshrathi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Things look better now, just check the comment.
And try creating a student locally.
Squash all your commits.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants