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
Facilitators can create workshops with regional partners #20881
Facilitators can create workshops with regional partners #20881
Conversation
b860130
to
ecbf985
Compare
New permissions for csf facilitators Hide delete butotn for facilitators
02158fa
to
659540c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Our client-side permissions are fairly scattered. I wonder if it would make sense to consolidate into the permissions class, adding properties like canCreateWorkshop
, canDeleteWorkshop
etc?
|
||
return ( | ||
<div> | ||
<h1>Your Workshops</h1> | ||
<ButtonToolbar> | ||
{(this.permission.isWorkshopAdmin || this.permission.isOrganizer) && | ||
{(this.permission.isWorkshopAdmin || this.permission.isOrganizer || this.permission.isCsfFacilitator) && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about calling this out above as const canCreate
, like we do for canDelete
, for consistency and readability?
Facilitators now have permissions to create workshops with regional partners