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

[#13174] Option to hide self responses when viewing responses #13175

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

Conversation

henriquepaes1
Copy link

Fixes #13174

Outline of Solution

The solution consists in adding a control variable in the session results screen. That was accomplished by creating a switch in the session-result-page component. The switch's status is then passed to the question-response-panel with the angular Input annotation. This variable is used as a conditional for self responses rendering.

The response view now has a switch controlling the self responses, as shown below:

image

When the switch isn't toggled, the student can view their own responses:

image

Once the switch is toggled:

image

The student can't see their own responses:

image

@henriquepaes1
Copy link
Author

I would like to know if would be useful to create tests for the feature's expected behavior (considering the switch for conditional rendering). In case of positive answer, I could use some guidance to write the corresponding tests, since I'm new to Jest and front-end testing in general.

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.

Student viewing responses: allow hiding self reponses
1 participant