Skip to content
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.

Sign up for GitHub

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

Jump to bottom

confidenceSlider excluded from answered question statements #26

Closed
danielghost opened this issue Sep 19, 2023 · 1 comment · Fixed by #27
Closed

confidenceSlider excluded from answered question statements #26

danielghost opened this issue Sep 19, 2023 · 1 comment · Fixed by #27
Assignees
@danielghost
Labels
bug released

Comments

@danielghost
Copy link
Contributor

The confidenceSlider component doesn't send a statement as it is missing from sendQuestionAnswered. The score and success should be excluded from the result.
@danielghost danielghost self-assigned this Sep 19, 2023
@danielghost danielghost changed the title confidenceSlider excluded from question _answered_ statements confidenceSlider excluded from answered question statements Sep 19, 2023
danielghost added a commit that referenced this issue Sep 20, 2023
@danielghost
Fix: added answered statement for the confidenceSlider (fixes #26).
107f728
@danielghost danielghost mentioned this issue Sep 20, 2023
Merged
danielghost added a commit that referenced this issue Sep 20, 2023
@danielghost
Fix: added _answered_ statement for the confidenceSlider - excludes `…
e6c4bdb 
…score` and `success` from the `result`. (fixes #26). (#27)
github-actions bot pushed a commit that referenced this issue Sep 20, 2023
@semantic-release-bot
Chore(release): 2.1.3 [skip ci]
9ec7891 
## [2.1.3](v2.1.2...v2.1.3) (2023-09-20)

### Fix

* added _answered_ statement for the confidenceSlider - excludes `score` and `success` from the `result`. (fixes #26). (#27) ([e6c4bdb](e6c4bdb)), closes [#26](#26) [#27](#27)
@github-actions
Copy link

🎉 This issue has been resolved in version 2.1.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danielghost danielghost

Labels
bug released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: added answered statement for the confidenceSlider (fixes #26). cgkineo/adapt-xAPI
1 participant
@danielghost