Skip to content
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

[Role/Permission] clean-up (4 of 4) #3540

Closed
wants to merge 5 commits into from

Conversation

ridz1208
Copy link
Collaborator

@ridz1208 ridz1208 commented Mar 7, 2018

This adds the Role infrastructure to LORIS. This is one of 4 PRs which will ultimately result in assigning roles to users. each role has affiliated permissions and automatically checks the proper permission when selected. This change will be mainly visible in the user_accounts module.

PART 4: SQL clean-up patches

in order to test, the following SQL inserts can be used:

INSERT INTO role (Name, Label) VALUES ('coordinator', 'Coordinator');
INSERT INTO role (Name, Label) VALUES ('data', 'Data-Entry');
INSERT INTO role (Name, Label) VALUES ('upload', 'Data-upload');


INSERT INTO role_permission_rel VALUES ((SELECT RoleID FROM role WHERE Name='coordinator'), (SELECT permID FROM permissions WHERE code='send_to_dcc'));
INSERT INTO role_permission_rel SELECT RoleID,permID FROM role JOIN permissions WHERE Name='data' AND code IN ('data_dict_view','data_entry','data_dict_edit','candidate_parameter_view');
INSERT INTO role_permission_rel SELECT RoleID,permID FROM role JOIN permissions WHERE Name='upload' AND code IN ('video_upload','media_write','media_read');

Original PR by @taracampbell : #2642
Updated/Standardized/DAO full version : ridz1208#9

@ridz1208 ridz1208 added Category: Feature PR or issue that aims to introduce a new feature [branch] minor labels Mar 7, 2018
@ridz1208 ridz1208 changed the title 2018 03 10 tar perm roles p4 [Role/Permission] clean-up (4 of 4) Mar 7, 2018
@ridz1208 ridz1208 added the State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed label Mar 7, 2018
@ridz1208 ridz1208 force-pushed the 2018_03_10_tar_perm_roles_p4 branch from f728f66 to 8394c87 Compare March 7, 2018 23:31
@driusan driusan added State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed and removed State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed labels Mar 12, 2018
@driusan
Copy link
Collaborator

driusan commented Mar 12, 2018

closing for same reason as #3538 (also re-added blocked, because I had accidentally removed it on the wrong PR.)

@driusan driusan closed this Mar 12, 2018
@ridz1208 ridz1208 mentioned this pull request Mar 31, 2021
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Feature PR or issue that aims to introduce a new feature State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants