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

Fix permissions in blog component #36

Open
7 tasks
paarals opened this issue Mar 29, 2021 · 2 comments
Open
7 tasks

Fix permissions in blog component #36

paarals opened this issue Mar 29, 2021 · 2 comments

Comments

@paarals
Copy link

paarals commented Mar 29, 2021

Els permisos a debat haurien de ser:

  • Comentar
  • Adhesió

Actualment, es mostren els permisos de propostes:
Link: https://test.encomuparticipa.barcelonaencomu.cat/admin/assemblies/assembleademo/components/59/permissions/edit

  • Endorse
  • Vote
  • Create
  • Withdrawn
  • Amend
bug_blog.mp4

és un error general de decidim

@paarals paarals changed the title Fix permissions in debates component Fix permissions in blog component Apr 9, 2021
@aitorlb aitorlb self-assigned this Apr 9, 2021
@aitorlb
Copy link

aitorlb commented Apr 13, 2021

Neither endorse nor comment actions can be blocked via permissions (authorization handler), even after setting them up at the component level in the code (component.actions), in decidim 0.23.

Regarding the endorse action, it does seem like a bug, cause the necessary logic is there but the permissions set up at the admin level are ignored so the verification modal is never shown. However, regarding the comment action, the comments module is lacking the logic to render the verification modal and this is why this action cannot be blocked at the admin level using permissions by default in any other module where resources can be commented.

I propose leaving comments out of the scope, since it's more like a whole feature request rather than a bug fix.

@microstudi
Copy link
Contributor

Actually it is possible to restrict comment using verifications. The full implementation is done in version 0.24. I propose upgrade to 0.24 and then re-evaluate this issue.

@aitorlb aitorlb removed their assignment Aug 26, 2021
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

No branches or pull requests

3 participants