-
Notifications
You must be signed in to change notification settings - Fork 291
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
Learning paths
: Hide unreleased learning objects in learning path view
#9105
Learning paths
: Hide unreleased learning objects in learning path view
#9105
Conversation
WalkthroughThe recent changes enhance the visibility and accessibility of learning objects in the Artemis platform. Key updates include the introduction of a method to check visibility, refined filtering in learning path logic, and UI enhancements to signify unreleased content. These modifications create a more intuitive user experience, ensuring students interact only with accessible materials while improving overall control flow in related services. Changes
Sequence Diagram(s)sequenceDiagram
participant Student
participant LearningPathService
participant LearningObjectService
participant UI
Student->>LearningPathService: Request learning path
LearningPathService->>LearningObjectService: Check visibility of learning objects
LearningObjectService-->>LearningPathService: Return visibility status
LearningPathService-->>UI: Send visible learning objects
UI->>Student: Display learning path with accessible objects
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
src/test/java/de/tum/in/www1/artemis/competency/CompetencyUtilService.java
Show resolved
Hide resolved
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.
tested on ts4, works as expected. code also LGTM
src/main/java/de/tum/in/www1/artemis/service/learningpath/LearningPathService.java
Outdated
Show resolved
Hide resolved
ce41610
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.
Actionable comments posted: 1
src/main/java/de/tum/in/www1/artemis/service/learningpath/LearningPathService.java
Outdated
Show resolved
Hide resolved
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.
Actionable comments posted: 1
d686eb6
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.
Actionable comments posted: 2
src/main/java/de/tum/in/www1/artemis/service/learningpath/LearningPathService.java
Show resolved
Hide resolved
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 after merge conflict
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
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 after merge conflicts
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 after merge conflicts
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.
Maintainer approved
Checklist
General
Server
Client
Motivation and Context
Learning paths should not show unreleased learning objects to the student, which is currently the case.
Description
Learning paths no longer send data like the title to the client. Instead the unreleased learning objects are marked with a placeholder in the learning path view, so students are aware that they will get more learning opportunities later on.
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
Test Coverage
Client
Server
Screenshots
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Tests