Resolve "[Queue-2.0] CSV Import for TAs"
What does this mr do?
Allow teachers to upload a CSV file with "netId","role"
into queue to add several students, ta's etc.
Screenshots
before
after
Error gets displayed when there is something wrong with the CSV
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 #396 (closed)
Note: A redo after messing up !436 (closed)
Merge request reports
Activity
changed milestone to %Queue 2.0 Initial deploy
added priohigh + 1 deleted label
requested review from @Lemaire and @otto
assigned to @cedricwilleken
- Resolved by Chris Lemaire
Copy from closed MR: I believe, in Submit, errors are accumulated and displayed together to immediately be able to resolve all issues as a teacher. It might be useful to do something like this in the future, but probably not necessary right now.
Looks like there is a small conflict in the gradle file. Could you still resolve this @cedricwilleken ?
added 12 commits
-
6567e034...8fb7438e - 11 commits from branch
development
- 727b73fb - Merge branch 'development' into 396-queue-2-0-csv-import-for-tas-1
-
6567e034...8fb7438e - 11 commits from branch
added 11 commits
- f31a63f8 - Add endpoint for uploading CSV for new participants
- e5d8d9d7 - Add error message incase something fails with the csv parsing
- 7907ea86 - Add jdoc to new methods
- 1e38e170 - Add test for error in csv
- 99208db4 - Add changelog entry
- 8e2bc40a - Add documentation to Edition controller
- 17558539 - Add CodeMirror and format CSV example for participants
- 8182cc84 - Add test to make sure CSV's get parsed correctly
- 81032624 - Fix import
- 7ffa125f - Run gradlespotless apply
- 690be815 - Add DATABASE_TO_UPPER=false option to h2 test datasource
Toggle commit listadded 1 commit
- d82836cc - Add DATABASE_TO_UPPER=false option to h2 test datasource
added 1 commit
- 4bce2573 - Add DATABASE_TO_UPPER=false option to h2 test datasource
mentioned in issue #418
added 45 commits
- cabfea4e - We've been in 2021 for a while now
- d4e83532 - Temp allow deployments from development branch
- af5da91b - fix gradle
- d314d03f - Attempt to deploy with sentry cli notifying Sentry of the new release
- 1d9dc39b - Fix commenting
- 20232024 - Remove license targets from build exclusion
- 6fc3cc03 - Temp disable license plugins
- 27f66afc - Update copyright year
- 412d3129 - Different sentry-cli command
- 189a1d65 - change sentry release config
- 478a338e - fix sentry things
- c2d2d218 - Fix yaml
- 41b76c5d - ensure organisation for sentry
- 5bd67bda - more sentry changes...
- 9ecff79e - Test
- 452cb033 - Try with the correct sentry server...
- 1cb7e4f4 - Add an initial version for Lab constraints
- 915f81fc - Add tests for LabController and fix small bugs
- 214722f8 - Add new table-setup additions: lab constraints
- 2daa6a53 - Add tests for the lab create DTOs
- ce7a085c - Add mail service for sending a Course create request
- 96adc41f - Add tests for new MailService and MailController
- 78ac1833 - Disallow Students to place feedback on TAs at exams
- 114b9548 - Add back feedback overview and fix input
- c947f8fd - Fix forwarded comments being visible to student
- 07022ac5 - Add tests and fix minor permission issue
- bfa222ac - Add database-platform setting to PostgreSQL test setup
- 8ecd6ccc - Change the generated value strategy for LabRequestConstraint
- 3fe2235c - Add create Module and create Assignment buttons
- 0282a6c6 - Add tests for the new module and assignment functionality
- c1178fbb - Add a standard sorting to the lab page
- b790db0a - Add filter for the type of the lab
- d6707837 - Add remove current filter button to labs page
- 4ea356fe - Add module filter to labs page
- 2527cf4e - Add some margins to the lab filters
- 107de398 - Add lab sort and filter to changelog file
- 7fe1ea16 - Add Completed badge to labs that are done
- 4da367ac - Upgrade Spring boot to 2.5.3
- 777d80fd - Add profile information to test application yaml
- 35f49f8d - Add access control for request course methods
- 012b64c4 - Fix permission and Jitsi room URL call
- 6407a7f9 - Add user information to Sentry
- 5aec5c5d - Ensure non-null RequestViewDTO 'timeSlot' field
- 06000ed6 - Add Sentry to logback
- eef87a1e - Run spotless
Toggle commit listadded 77 commits
-
eef87a1e...5295fcb4 - 76 commits from branch
development
- c3ae69a6 - Merge branch 'development' into 396-queue-2-0-csv-import-for-tas-1
-
eef87a1e...5295fcb4 - 76 commits from branch
added 25 commits
-
ead793ba...b83135f2 - 24 commits from branch
development
- 7a702315 - Merge branch 'development' into 396-queue-2-0-csv-import-for-tas-1
-
ead793ba...b83135f2 - 24 commits from branch
- Resolved by Otto Visser
Could you quickly resolve all conflicts with this one so that we can merge it before the big boy !456 (merged) ?
added 11 commits
-
7a702315...bc88c2d3 - 10 commits from branch
development
- 7c161b7c - Merge branch 'development' into 396-queue-2-0-csv-import-for-tas-1
-
7a702315...bc88c2d3 - 10 commits from branch