-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Implement Database, Architecture and Permission System #623
Comments
@aditya1702 do you have any idea how we can prepare a permission list? |
@rafalkowalski I think we can make use of some frameworks of Flask to achieve this. I will look into it. |
@aditya1702 can i take care of permissions? |
@rafalkowalski Good idea. Please do. |
@mariobehling can track-organizer edit every tracks or track defined by organizer/co-organizer? |
@rafalkowalski A Track Organizer would be specific for an Event, i.e. a user can be TO in an Event where he has been assigned as one (by the event creator). The TO should be permitted to edit tracks of only those events where he is a TO in. |
@rafalkowalski Have you started working on the issue, I would like to help. Or I can take up the issue if you want, you seem to be assigned to a lot of issues. |
@shivamMg I am also working on the issue. You can help if u want 😄 |
@aditya1702 Would love to help. Are you working on a common branch? 😃 |
do you have an idea @shivamMg ? |
A track organizer needs to be assigned to a specific track to be able to edit the track. But, a track organizer can also be added to several tracks. Could you please document a list what each role is allowed to do. |
@shivamMg @aditya1702 Permissions |
@rafalkowalski Please also regard the activity of the project, number of contributors and latest dates of commits. |
@mariobehling We are working on a custom solution for this. Extensions might require a lot of tinkering to make them suitable for our purpose. |
@mariobehling custom solution will be the best option for us, because our permission model isn't a standard |
@rafalkowalski Ok, makes sense. Thank you! |
This issue is really starting to be urgent as it is blocking other features. |
I put some thought into the Speaker role, and we might not need it. The Speaker role requires two permissions, one editing his own info (Speaker model he created) and other reading the Session he sent proposal for. After a user sends a Speaker proposal we can associate that Speaker model with the user. And after it has been accepted, only the associated user will be given permission to edit the info, so restricting access only to that user. And since we already know what Session a Speaker model is related to, we can define reading restrictions for it. As for why Speaker can't be an event specific role, it requires access to just a specific instance of a service (speaker) inside an event. Where as we define permissions for a service as a whole. So we will implement the SPEAKER role using the Speaker model itself. |
Please check if other use cases are relevant here too e.g.:
|
But the speakers are not allowed to edit sessions. They can only see their own sessions. Quoting from the main description above:
|
Yes, speakers can change for example their abstract. And also if several speakers have the same session they can all edit the session. |
Implement Speaker role though models: #955 |
Sitewide Roles
Event Specific Roles
SPEAKERImplement Database, Architecture and Permission System #623 (comment)The text was updated successfully, but these errors were encountered: