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] Student viewing responses: allow hiding self responses #13182

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

Conversation

Zijian-Yang
Copy link

@Zijian-Yang Zijian-Yang commented Oct 19, 2024

Fixes #13174
This pull request implements the feature to allow students to hide their own responses when viewing feedback session results. The changes include:

  1. Added a toggle switch in the session results page for students to hide/show their own responses.
  2. Implemented the logic to filter out self-responses when the toggle is activated.
  3. Updated the UI to reflect the hidden/shown state of self-responses.
  4. Added unit tests to ensure the new feature works as expected.

These changes address the user request to de-clutter the view by providing an option to remove their own responses from the display. The implementation follows the suggestion outlined in the original issue.

This PR builds upon #13175, which initially proposed this feature.

The changes have been tested locally and all existing tests pass. New tests have been added to cover the new functionality.
Please review the changes and let me know if any modifications or additional information is needed.

Copy link

Hi @Zijian-Yang, thank you for your interest in contributing to TEAMMATES!
However, your PR does not appear to follow our contribution guidelines:

  • Description must reference the issue number the PR is fixing, e.g. Fixes #<issue-number> (or Part of #<issue-number> if the PR does not address the issue fully)

Please address the above before we proceed to review your PR.

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
2 participants