Draft: UI component for allowed requests for students with multiple roles
What does this mr do?
This MR enables people with both staff and student roles in a shared lab to differentiate between the assignments they can enqueue for vs the assignments they can take requests for in the lab enqueueing page. The affected UI component is highlighted in a picture below.
Note: this change only applies for persons with multiple roles (ie. both staff and student roles) in one lab.
UI change suggested by @rwbackx .
Screenshots
before
In the old version, only the allowed assignments one could enqueue for were shown

after
In the new version, all assignments associated with the session are shown. Moreover, for people with multiple roles, the assignments for which they can enqueue for versus take requests for are highlighted in different sections.

Does this MR meet the acceptance criteria?
-
I have added a changelog entry to reflect the significant changes I made. -
Tests were created to test the feature. -
I have updated the documentation accordingly. -
I adhere to the style guide.
Closes #571 (closed)
Merge request reports
Activity
added difficultymedium priomed labels
Oana Milchi, this merge request has policy violations and errors. To unblock this merge request, fix these items:
- Resolve all violations in the following merge request approval policies: Security check.
- Resolve the errors and re-run the pipeline. If you think these items shouldn't be violations, ask eligible approvers of each policy to approve this merge request.
Errors
- There is a mismatch between the scans of the source and target pipelines. The following scans are missing: Dast, Sast, Secret detection
Comparison pipelines
- Target branch (
development): None - Source branch (
571-allowed-assignments-for-multiple-roles): #1227225
Edited by GitLab Security Botadded 60 commits
-
9a74b8e1...b09c2582 - 27 commits from branch
development - b09c2582...0cf7a592 - 23 earlier commits
- 60465c4d - Merge branch 'development' into 'master'
- e85167b1 - Merge branch 'development' into 'master'
- 88848c2a - Merge branch 'development' into 'master'
- 8eea5237 - Update lab-request-info.html
- d236f7ae - Merge branch 'owvisser-master-patch-54911' into 'master'
- 26226430 - Merge branch 'development' into 'master'
- 8f16f735 - Merge branch 'development' into 'master'
- a43ed84a - Merge branch 'development' into 'master'
- ca56a1a5 - Merge branch 'development' into 'main'
- b830ae50 - Merge branch 'main' into 571-allowed-assignments-for-multiple-roles
Toggle commit list-
9a74b8e1...b09c2582 - 27 commits from branch
