-
Notifications
You must be signed in to change notification settings - Fork 26
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Bug?fix: template permissions #3398
馃悰 Bug?fix: template permissions #3398
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3398 +/- ##
========================================
+ Coverage 79.9% 83.0% +3.1%
========================================
Files 816 816
Lines 34583 34583
Branches 1362 1362
========================================
+ Hits 27663 28737 +1074
+ Misses 6752 5665 -1087
- Partials 168 181 +13
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
馃帀
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.
馃憤
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.
Great! Luckily you've fixed before someone complained. (:
Kudos, SonarCloud Quality Gate passed!聽 聽 0 Bugs No Coverage information |
What do these changes do?
Study and Template
Collaborator
s have permission for editing the metadata: title, description, TSR, accessRights...When sharing a Study or Template using the frontend, by default, they get
Collaborator
role. This might be convenient for Studies, but not for Templates. This PR makes the default role to beViewer
for Templates.Before:
After:
Bonus:
Related issue/s
How to test
Checklist