-
Notifications
You must be signed in to change notification settings - Fork 13
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
[INF-227] Enforce correct response value complies with base type #2433
[INF-227] Enforce correct response value complies with base type #2433
Conversation
views/js/qtiCreator/widgets/interactions/textEntryInteraction/states/Correct.js
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.
- New code is covered by tests (if applicable)
- Tests are running successfully (old and new ones) on my local machine (if applicable)
- New code is respecting code style rules
- New code is respecting best practices
- New code is not subject to concurrency issues (if applicable)
- Feature is working correctly on my local machine (if applicable)
- Acceptance criteria are respected
- Pull request title and description are meaningful
- Pull request's target is not
master
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.
Before my approval, can you check why ci is failing?
Front-end summary Node 18
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2433 +/- ##
==========================================
Coverage 16.07% 16.07%
Complexity 4207 4207
==========================================
Files 391 391
Lines 13486 13486
==========================================
Hits 2168 2168
Misses 11318 11318 ☔ View full report in Codecov by Sentry. |
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.
LGTM!
Version
There are 0 BREAKING CHANGE, 0 feature, 16 fixes |
Ticket: INF-227
What's Changed
Demo
How to test
Kitchen