wekan / wekan

The Open Source kanban (built with Meteor). Keep variable/table/field names camelCase. For translations, only add Pull Request changes to wekan/i18n/en.i18n.json , other translations are done at https://app.transifex.com/wekan/wekan only.
https://wekan.github.io
MIT License
19.6k stars 2.83k forks source link

Remove Archive and Duplicate Buttons from cards in board view #2599

Closed derbolle closed 5 years ago

derbolle commented 5 years ago

Issue

Add these issues to elsewhere:

Other Wekan issues can be added here.

Server Setup Information:

Problem description:

How it is now: Desktop Browser: grafik

Mobile View: grafik

How it should work:

loucurapt commented 5 years ago

@xet7 Not everything reported in this issue has been fixed, so I don't think it should be closed.

The Archive button still appears in the "My Boards" page in all cards. even if I'm not the administrator of the board, and clicking the button displays no confirmation. The Duplicate button also doesn't show confirmation.

xet7 commented 5 years ago

@loucurapt

Moved to #2644

loucurapt commented 5 years ago

@xet7 Thank you! Don't forget the permissions check.

xet7 commented 5 years ago

@loucurapt

What permission checks? Please add new issue about those, I don't know what is missing.

loucurapt commented 5 years ago

@xet7 It's one of the things that @derbolle mentioned and I reiterated. The Archive button is displayed on all boards in the main page, even if I'm not an admin of the board. I'm also an administrator user though (in the admin panel)... could that be it?

I can report a new issue if you like.

xet7 commented 5 years ago

You are Admin User, so you see those buttons. Those users that are not Admin or BoardAdmin at all, have those hidden. So there is not permission issue at all, this is how it currently works. If you want those hidden, create new user that has only user permissions, not any Admin/BoardAdmin permission. Don't add new issue about this.

loucurapt commented 5 years ago

Ok. Thank you for the clarification! It makes sense