Skip to content

feat: add never_but_include_grade visibility option #2489

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

feat: add never_but_include_grade visibility option #2489

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
060549b
feat: add 'never show score but count in final grade' visibility option
Sep 16, 2025
eec86aa
test: add test
Anas12091101 Sep 29, 2025
760f31f
fix: lint issues
Anas12091101 Sep 30, 2025
8adc1b7
fix: option title and description and add message desctription
Anas12091101 Oct 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/generic/configure-modal/ConfigureModal.test.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,8 @@ describe('<ConfigureModal /> for Subsection', () => {
expect(getByText(messages.neverShowAssessmentResultsDescription.defaultMessage)).toBeInTheDocument();
expect(getByText(messages.showAssessmentResultsPastDue.defaultMessage)).toBeInTheDocument();
expect(getByText(messages.showAssessmentResultsPastDueDescription.defaultMessage)).toBeInTheDocument();
expect(getByText(messages.neverShowAssessmentResultsButIncludeGrade.defaultMessage)).toBeInTheDocument();
expect(getByText(messages.neverShowAssessmentResultsButIncludeGradeDescription.defaultMessage)).toBeInTheDocument();
});

it('switches to the subsection Advanced tab and renders correctly', async () => {
Expand Down
6 changes: 6 additions & 0 deletions src/generic/configure-modal/VisibilityTab.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,12 @@ const VisibilityTab = ({
<FormattedMessage {...messages.showAssessmentResultsPastDue} />
</Form.Radio>
<Form.Text><FormattedMessage {...messages.showAssessmentResultsPastDueDescription} /></Form.Text>
<Form.Radio value="never_but_include_grade">
<FormattedMessage {...messages.neverShowAssessmentResultsButIncludeGrade} />
</Form.Radio>
<Form.Text>
<FormattedMessage {...messages.neverShowAssessmentResultsButIncludeGradeDescription} />
</Form.Text>
</Form.RadioSet>
</>
) : (
Expand Down
16 changes: 16 additions & 0 deletions src/generic/configure-modal/messages.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,26 +170,42 @@ const messages = defineMessages({
alwaysShowAssessmentResults: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.always-show-assessment-results',
defaultMessage: 'Always show assessment results',
description: 'Always show assessment results option in visibility tab',
},
alwaysShowAssessmentResultsDescription: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.always-show-assessment-results-description',
defaultMessage: 'When learners submit an answer to an assessment, they immediately see whether the answer is correct or incorrect, and the score received.',
description: 'Always show assessment results option description in visibility tab',
},
neverShowAssessmentResults: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.never-show-assessment-results',
defaultMessage: 'Never show assessment results',
description: 'Never show assessment results option in visibility tab',
},
neverShowAssessmentResultsDescription: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.never-show-assessment-results-description',
defaultMessage: 'Learners never see whether their answers to assessments are correct or incorrect, nor the score received.',
description: 'Never show assessment results option description in visibility tab',
},
showAssessmentResultsPastDue: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.show-assessment-results-past-due',
defaultMessage: 'Show assessment results when subsection is past due',
description: 'Show assessment results past due option in visibility tab',
},
showAssessmentResultsPastDueDescription: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.show-assessment-results-past-due-description',
defaultMessage: 'Learners do not see whether their answer to assessments were correct or incorrect, nor the score received, until after the due date for the subsection has passed. If the subsection does not have a due date, learners always see their scores when they submit answers to assessments.',
description: 'Show assessment results past due option description in visibility tab',
},
neverShowAssessmentResultsButIncludeGrade: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.never-show-assessment-results-but-include-grade',
defaultMessage: 'Never show individual assessment results, but show overall assessment results after due date',
description: 'Never show assessment results but include in grade calculations option in visibility tab',
},
neverShowAssessmentResultsButIncludeGradeDescription: {
id: 'course-authoring.course-outline.configure-modal.visibility-tab.never-show-assessment-results-but-include-grade-description',
defaultMessage: 'Learners do not see question-level correctness or scores before or after the due date. However, once the due date passes, they can see their overall score for the subsection on the Progress page.',
description: 'Never show assessment results but include in grade calculations option description in visibility tab',
},
setSpecialExam: {
id: 'course-authoring.course-outline.configure-modal.advanced-tab.set-special-exam',
Expand Down