Skip to content

Conversation

@zarifmahmud
Copy link
Contributor

@zarifmahmud zarifmahmud commented Jul 6, 2022

Details

  • I was getting a bug where if a board is supposed to be anonymous, refreshing the page causes some posts to be anonymous and others to have the correct name, like in this picture:

image

To recreate:

  1. Create a new board using teacher, and have a student account join it.
  2. Make some posts with the student account.
  3. Teacher account unchecks the permission boxes that allows teachers and students to see post name, and updates the configuration. (this can also be done in the very beginning)
  4. Then if you access the board from any account (teacher, student who made the posts, or another student), and keep refreshing, you'll reach a state where one or more of the posts aren't anonymized. Also, refreshing the page can lead to an author's name being shown for a split second before it switches to anonymous.
  • This fix is meant to fix the bug and the author name leak.

@zarifmahmud
Copy link
Contributor Author

The fix is a bit weird, since I'm adding an optional "authors" param to the PostOptions, but the thinking behind it is when a board is first loaded it will now by default show author names as "Anonymous", and then later on in configureBoard() when this.setAuthorVisibilityAll() is called, the author names can be loaded.

@zarifmahmud zarifmahmud requested a review from a team July 6, 2022 19:39
Copy link
Contributor

@JoelWiebe JoelWiebe left a comment

Choose a reason for hiding this comment

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

Successful tests:

  • Adding/removing anonymity privileges makes immediate change on both teacher and student canvases
  • Refreshing the browser maintains anonymity

Additional items/questions

  • posts in lists and buckets do not appear to get anonymized
  • do you think it makes sense that users should still be able to see display name for their own posts and anonymize only others' posts (this way if they wanted to locate their posts they could)

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.

3 participants