-
Notifications
You must be signed in to change notification settings - Fork 39
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
Feature request: Allow voting in read-only rooms #46
Comments
this is an interesting use case. for v2 I think the only issue is with app user's permissions, since its him who create and edit the message to show the votes, that might be the problem with #33 as well. need a proper test and if that's the case just need to document it somewhere (and maybe give the user an alert about it) ;) |
The problem is most likely due to Poll user rights. I made it admin and was able to create a vote in a read-only room. Sorry for my English. |
it definitely is the app role permission.. give it the permission to post in read-only and you are good to go.. no need to give it admin |
thanks @awunder for taking a look .. I didn't have time to proper look into this. Even if I can't fix these errors, I need to add some error handling, show somewhere the error message if an error occurred |
yeah, some user feedback would be great.. i "figured out" why it wont post the poll via issue search and finding this issue :) |
@sampaiodiego I haven't looked into the underlying stuff, maybe this is nigh on impossible - or maybe it should be working and there's a bug!
Let me know what you think 😄
The text was updated successfully, but these errors were encountered: