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

[1.2.0 RC2] Anonymous poll not saved #829

Closed
Dennis1993 opened this issue Feb 15, 2020 · 3 comments · Fixed by #838
Closed

[1.2.0 RC2] Anonymous poll not saved #829

Dennis1993 opened this issue Feb 15, 2020 · 3 comments · Fixed by #838
Assignees
Labels
Milestone

Comments

@Dennis1993
Copy link

If I check the checkbox "anonymous poll" and "hide names" the names will be hide. That works. If I reload the page the checkbox is not checked anymore but the function is working. It looks like the current status is not loaded correctly to check the checkbox if its activated

Steps to reproduce:

  1. create a new poll
  2. make is anonymous and hide names
  3. reload the page an see the settings of it

image

@Dennis1993 Dennis1993 added the bug label Feb 15, 2020
@dartcafe
Copy link
Collaborator

The full anonymous setting has no further function and will be removed in the next version.

@dartcafe dartcafe added this to the 1.4 milestone Feb 16, 2020
@v1r0x
Copy link
Collaborator

v1r0x commented Feb 16, 2020

@dartcafe Any reason why?

@dartcafe
Copy link
Collaborator

@v1r0x
It makes no sense in the end. Owner and admin users could remove this setting and see the names. So it is is a misleading feature IMHO and prentents to give anonymity.

Anonymous polls are now anonymous to everyone when the anoymous flag is set. Admin users are treated as a normal user, unless he is the poll owner or the owner gave the permission to admins to edit the poll.

BTW: There is a bug in setting the edit permission for admins, if the owner is an admin user. Will fix this, when back from vacation (this evening).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants