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.

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

Jump to bottom

Sessions: instructor view results: stop showing a random identifier for anonymous responses #8532

Open
bqnguyen94 opened this issue Feb 24, 2018 · 1 comment
Open

Sessions: instructor view results: stop showing a random identifier for anonymous responses #8532

bqnguyen94 opened this issue Feb 24, 2018 · 1 comment
Labels
a-Security Protection from security threats s.ToInvestigate The issue sounds complete but needs validation by a core team member

Comments

@bqnguyen94
Copy link
Contributor

As per #6092, we need to stop displaying the random hash in anonymised participants' name, e.g. Anonymous student 1234567890 to Anonymous student. #8443 addresses this by removing the hash before data is rendered, but only for student view; instructor users can still see the trailing hash and could trace the anonymised student. Ideally, we should remove the hash completely from FeedbackSessionResultsBundle.
@whipermr5 whipermr5 changed the title FeedbackSessionResultsBundle: remove anonymous participant's hash identifier Sessions: instructor view results: stop showing a random identifier for anonymous responses Mar 9, 2018
@madanalogy madanalogy added the help wanted Moderate difficulty, small localized change; suitable for novice contributors label Sep 25, 2020
@halfwhole halfwhole mentioned this issue Dec 21, 2020
Merged
@madanalogy madanalogy added a-Security Protection from security threats t-Java labels Mar 4, 2021
madanalogy added a commit that referenced this issue Mar 11, 2021
@halfwhole @madanalogy
[#8532] Sessions: instructor view results: stop showing a random iden…
a8a1834 
…tifier for anonymous responses (#10890)

Co-authored-by: Ahmed Bahajjaj <Ahmed_Bahajjaj@u.nus.edu>
@undermyumbrella1 undermyumbrella1 mentioned this issue Apr 12, 2023
Closed
@undermyumbrella1
Copy link
Contributor

Hi I have created a pr to try to fix this issue and possibly discuss if my approach is ok. May I request for some help reviewing it? Thank you!

@wkurniawan07 wkurniawan07 added s.ToInvestigate The issue sounds complete but needs validation by a core team member and removed help wanted Moderate difficulty, small localized change; suitable for novice contributors labels Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
a-Security Protection from security threats s.ToInvestigate The issue sounds complete but needs validation by a core team member
Projects
None yet
Milestone
No milestone
5 participants
@whipermr5 @wkurniawan07 @bqnguyen94 @madanalogy @undermyumbrella1