Skip to content

Conversation

@yl9431
Copy link
Contributor

@yl9431 yl9431 commented Nov 18, 2025

Closes: #1057
Closes: #1058

…isplay of error and account for other errors
…ight before submitting that doesn't seem like a good idea
@yl9431 yl9431 changed the title better training answers better training answers + all questions answered Nov 18, 2025
@yl9431 yl9431 changed the title better training answers + all questions answered display # of answers for checkboxes + all questions answered Nov 18, 2025
@yl9431 yl9431 requested a review from ZZwatermelon November 24, 2025 18:25
variant="contained"
sx={{ alignSelf: "flex-end" }}
disabled={module.isLocked ?? false}
disabled={(module.isLocked || blockSubmit) ?? false}
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think block submit needs to be in a state or useEffect, I'm pretty sure you can just use answerSheet.some() in this conditional (or put that in a function and call that function here).

}
};

function editQuizOptions(quiz: TrainingModuleItem[]){
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this function can be named better, its a little confusing

@yl9431 yl9431 requested a review from ZZwatermelon December 2, 2025 18:19
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.

Set number of right answers for multi-answer Make it so you cannot submit partial trainings.

3 participants