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.

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

chore: update messages #99

Merged
merged 2 commits into from
Dec 6, 2023
Merged

chore: update messages #99

merged 2 commits into from
Dec 6, 2023

Conversation

leangseu-edx
Copy link
Contributor

@leangseu-edx leangseu-edx commented Dec 5, 2023

  • update message from the UAT docs
  • xblock honor show rubric setting

Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (bdb6e51) 12.09% compared to head (b2e0347) 12.05%.

❗ Current head b2e0347 differs from pull request most recent head 5625bb0. Consider uploading reports for the commit 5625bb0 to get more accurate results

Files Patch % Lines
...ssessment/EditableAssessment/AssessmentActions.jsx 0.00% 2 Missing ⚠️
src/views/XBlockView/index.jsx 0.00% 2 Missing ⚠️
src/components/Assessment/messages.js 0.00% 1 Missing ⚠️
src/views/AssessmentView/index.jsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
- Coverage   12.09%   12.05%   -0.05%     
==========================================
  Files         137      137              
  Lines        1480     1485       +5     
  Branches      334      336       +2     
==========================================
  Hits          179      179              
- Misses       1213     1218       +5     
  Partials       88       88              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@muselesscreator muselesscreator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@leangseu-edx leangseu-edx merged commit de12d96 into master Dec 6, 2023
3 checks passed
@leangseu-edx leangseu-edx deleted the lk/uat-second-round branch December 6, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants