General: Support lti learning activities for multiple exercises
#10576
+211
−84
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lti-learning-activities-for-multiple-exercises
…lti-learning-activities-for-multiple-exercises
…lti-learning-activities-for-multiple-exercises
…ctivities-for-multiple-exercises' into chore/lti/support-lti-learning-activities-for-multiple-exercises
raffifasaro
dismissed stale reviews from KonstiAnon, Jan-Thurner, and coderabbitai[bot]
via
fe0973d
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
az108
previously approved these changes
Mar 26, 2025
…-for-multiple-exercises
…lti-learning-activities-for-multiple-exercises # Conflicts: # src/main/webapp/app/core/course/overview/course-exercises/course-exercises.component.ts # src/test/javascript/spec/component/course/course-exercises.component.spec.ts
raffifasaro
dismissed stale reviews from KonstiAnon, az108, Jan-Thurner, and coderabbitai[bot]
via
94a9259
helios-aet
bot
temporarily deployed
to
artemis-test3.artemis.cit.tum.de
There was a problem hiding this comment.
Followed the testing steps, tested on ts3
The contrast is very low here (step 8) - I had a pretty hard time reading something there, can we change that? Was that part of this PR (maybe add a PR description 😉 )?
I am not familiar with the LTI feature, how do I get to the view described in step 11? 😅
End-to-End (E2E) Test Results Summary
|
|||||||||||||||||||||||||||||||||||||||
az108
approved these changes
Mar 27, 2025
Jan-Thurner
approved these changes
Mar 27, 2025
…-for-multiple-exercises
End-to-End (E2E) Test Results Summary
|
|||||||||||||||||||||||||||||||||||||||
helios-aet
bot
temporarily deployed
to
artemis-test3.artemis.cit.tum.de
…-for-multiple-exercises
End-to-End (E2E) Test Results Summary
|
|||||||||||||||||||||||||||||||||||||||
helios-aet
bot
temporarily deployed
to
artemis-test3.artemis.cit.tum.de
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
General
Server
Client
Motivation and Context
Currently, instructors can only link learning activities to single exercises. This PR enables instructors to group multiple exercises into one learning activity to reflect an exercise week, for example.
Description
This pull request introduces a new feature for grouping exercises in the content selection table. When activated, Moodle creates a new learning activity that links to the earliest selected exercise. Additionally, a sidebar will display all other exercises chosen by the instructor within the group.
Steps for Testing
Prerequisites:
Make sure you are logged in to Artemis with a Test User 16)
Navigate to Moodle and login with the same test user (Test User 16}, same PW as for the Artemis Test Server)
Go to My Courses and open the lti test course
Activate Edit mode in top right corner
Add a learning activity by clicking on the blue + in "Test Topic"
Select Linhuber TS3
Click on "Select content"
Select the LTI test course
Select multiple exercises and activate the grouping
If the content was selected successfully, scroll down and hit "Save and display"
(If you get an error after linking, this is due to server sync, just try again by pressing the back button in your browser)
You should be able to see the grouped exercises view with the sidebar showing all selected exercises
Testserver States
You can manage test servers using Helios. Check environment statuses in the environment list. To deploy to a test server, go to the CI/CD page, find your PR or branch, and trigger the deployment.
Review Progress
Code Review
Manual Tests
Test Coverage
Screenshots
Grouped exercise view within Moodle:
New grouping feature in content selection table:
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Style