-
Notifications
You must be signed in to change notification settings - Fork 297
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
Adaptive learning
: Add repeated tests to learning paths
#9738
base: feature/adaptive-learning/learner-profile-graded-exercises
Are you sure you want to change the base?
Conversation
…/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests # Conflicts: # src/test/java/de/tum/cit/aet/artemis/atlas/service/LearningPathServiceTest.java
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
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.
I checked with an unrelated PR (#9850) and the issue is also present there, so it is not introduced by my current PRs. Could you create an issue for this problem? |
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.
Apart from the issues mentioned on #9699 code lgtm and works fine on TS5
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
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.
re-approve
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.
Reapprove
…ses' into feature/adaptive-learning/learner-profile-repeated-tests
Checklist
General
Server
Motivation and Context
Stacked on #9699
#8760
Description
Adapted scheduling of exercises in the learning path to respect the preferences regarding repeated tests.
This PR does not add any UI!
Steps for Testing
Prerequisites:
Testserver States
Note
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.
Review Progress
Performance Review
Code Review
Manual Tests
Performance Tests
Test Coverage
Summary by CodeRabbit
New Features
CourseLearnerProfile
andLearnerProfile
entities for managing learner profiles in relation to courses.Bug Fixes
Documentation
Tests