-
Notifications
You must be signed in to change notification settings - Fork 288
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
Development
: Disable LTI exercise link button when no exercise is selected
#9077
Development
: Disable LTI exercise link button when no exercise is selected
#9077
Conversation
WalkthroughThe recent changes enhance the LTI 1.3 deep linking functionality by updating the user interface to conditionally render buttons based on exercise selection, ensuring users are guided correctly. The backend logic now verifies selected exercises before initiating the deep link request and displays appropriate error messages if none are selected. Additionally, localization improvements were made for both English and German interfaces. Test cases were updated to reflect these new behaviors and ensure robust functionality. Changes
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 (
|
Bugfix
: Disable Moodle Exercise-Link button when no exercise is selected
src/test/javascript/spec/component/lti/lti13-deep-linking.component.spec.ts
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.
Code
Bugfix
: Disable Moodle Exercise-Link button when no exercise is selectedGeneral
: Disable Moodle Exercise-Link button when no exercise is selected
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.
Code, as I'm not a moodle admin
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 ts3 - working
General
: Disable Moodle Exercise-Link button when no exercise is selectedDevelopment
: Disable LTI exercise link button when no exercise is selected
Checklist
General
Client
Changes affecting Programming Exercises
Motivation and Context
When linking an Artemis exercise to Moodle, the instructor selects all those exercises from a list. The Link button is available without any selection though and the checkmarks are hardly visible.
Description
I disabled the button when no exercises are selected and changed the button text to highlight the necessary user action.
Steps for Testing
Full System Test
Prerequisites:
Artemis-Only Test
This test will throw internal server errors as Moodle context is missing. However, all changed aspects are visible here and no Moodle Account is required.
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
Screenshots
Summary by CodeRabbit
New Features
Localization
Tests