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.

Sign up for GitHub

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

Jump to bottom

Team verwalten: Rollen #1416

Merged
merged 10 commits into from
Jul 14, 2021
Merged

Team verwalten: Rollen #1416

merged 10 commits into from
Jul 14, 2021

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented May 14, 2021 •
edited
Loading

For Issue #1415

I will add commits until the features currently in the PR are approved. -> this PR is finished
Currently in this PR from the Issue:

  • Gast hinzufügbar machen

Aktivität Detail ansicht

  • Guest
    • Layout bearbeiten Knopf ausblenden/disablen
    • Klick auf Category -> Category Edit ausblenden
    • Wenn man auf den Activity Namen klickt wird dieser editierbar -> deaktivieren
    • Location Text disablen
    • ScheduleEntry Hinzufügen/löschen etc ausblenden -> needs Activity: edit/create/delete ScheduleEntry #1379 to be merged
    • Responsible DropDown disablen
    • ContentNodes: Icon ändern disablen
    • ContentNodes: Menu entfernen
    • ContentNodes: Textareas disablen
    • ContentNodes: Material + MaterialList disablen
    • ContentNodes: LAThematicArea disablen
    • ContentNodes: StoryConcept disablen (Textareas, drag/drop, löschen, hinzufügen)

Story

  • Guest
    • Editierbar Knopf disablen

Team

  • Guest/Member
    • Dropdown für Rolle disablen
    • Deactivate Knopf entfernen (ausser bei eigenen)
    • Resend Email Knopf entfernen
    • Invite Area entfernen
    • Delete Knopf entfernen
  • Manager
    • Wenn nur noch ein Manager im Team: Dropdown für Manager disablen, deactivate Knopf disablen, beides mit Tooltip: "Es muss mindestens ein Manager im Lager verbleiben"

Materials

  • Guest
    • Felder in Materiallist disablen (Auch Menu für löschen)
    • Hinzufügen row entfernen

Print

  • Nothing to do

Admin

  • Guest/Member
    • Textfelder disablen
    • Periods editieren disablen (add knopf weg, edit + menu weg
    • Danger Zone nicht anzeigen Hide danger zone if not manager #1382
    • Activity Categories: add weg, Edit Knopf bei der Category weg, Edit Layout weg, Delete weg
    • Material-list: create/edit/delete weg

For next PR's:

  • Make UX for disabled Links better:
    Remove hover on disabled links -> e-href component
    Remove hover on disabled buttons -> e-button

  • Make Picasso readonly for Guests

@BacLuc BacLuc mentioned this pull request May 14, 2021
Closed
33 tasks
pmattmann
pmattmann previously approved these changes May 15, 2021
View reviewed changes
@BacLuc BacLuc force-pushed the frontend-acl branch 2 times, most recently from 469aad3 to 56b73ff Compare June 6, 2021 15:14
@BacLuc
frontend/Story.vue: disable editable switch if user is guest
7b0eec8 
Pass camp as function instead of object into campRoleMixin.

Issue: ecamp#1415
@BacLuc BacLuc requested a review from pmattmann June 6, 2021 15:51
@BacLuc
frontend/Collaborators.vue: enable edit functions only for manager
180455c 
But allow deactivating your own CampCollaboration, as long as
you are not the last manager.
Prohibit the removal of the last manager.
In this view, removing the buttons didn't look well, so i disabled them.

Issue: ecamp#1415
@BacLuc
frontend/CollaboratorListItem.vue: add tooltip to explain that you ca…
862e9cf 
…nnot delete the last manager

I could not use the component directly as activator, because
it is disabled when the tooltip should be shown, and then it does
not activate the tooltip.

Issue: ecamp#1415
@BacLuc
frontend/PeriodMaterialLists.vue: disable if user is guest
0dc335e 
Issue: ecamp#1415
@BacLuc
frontend/Admin.vue: disable if user is not manager
7cafc29 
Issue: ecamp#1415
@BacLuc
backend/content-type: allow guest to fetch the entities of content-nodes
81de17a 
In these content-types, the ACL is defined in the Module.php.
Adapt them according to 7ff466d.

Issue: ecamp#1415
carlobeltrame
carlobeltrame approved these changes Jun 9, 2021
View reviewed changes
Copy link
Member

@carlobeltrame carlobeltrame left a comment

Choose a reason for hiding this comment

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

Sieht super aus, merci!

In den Camps die von den Seeds generiert werden ist niemand Manager... ist das schon lange so?

frontend/src/components/camp/CollaboratorListItem.vue
Comment on lines +30 to +31
v-bind="attrs"
v-on="on">
Copy link
Member

Choose a reason for hiding this comment

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

Die direkt aufs api-select bzw. den button-delete zu machen funktioniert nicht?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Leider nein, weil die disablen sind in dem fall, in dem das popup angezeigt wird.

@carlobeltrame
Copy link
Member

Vielleicht sollten wir dann irgendwann noch das Role Change Dropdown in einen Dialog auslagern oder eine Bestätigung einfügen... Man kann sich aktuell sehr schnell aus einem Lager ausschliessen.

@BacLuc
Copy link
Contributor Author

BacLuc commented Jun 10, 2021

Sieht super aus, merci!

In den Camps die von den Seeds generiert werden ist niemand Manager... ist das schon lange so?

Oh, das ist mir nicht aufgefallen. Soll ich das in dem PR noch machen, oder in einem anderen?

@carlobeltrame
Copy link
Member

Oh, das ist mir nicht aufgefallen. Soll ich das in dem PR noch machen, oder in einem anderen?

Das hast nicht du verbrochen, von mir aus kann der PR gemerged werden. Ausser du willst nochmals ein Review von @pmattmann für die neuen Commits.

@BacLuc
Copy link
Contributor Author

BacLuc commented Jun 10, 2021

Oh, das ist mir nicht aufgefallen. Soll ich das in dem PR noch machen, oder in einem anderen?

Das hast nicht du verbrochen, von mir aus kann der PR gemerged werden. Ausser du willst nochmals ein Review von @pmattmann für die neuen Commits.

Ein zweites Review wäre schon wichtig, er hat eigentlich noch nichts gereviewed.

@carlobeltrame carlobeltrame dismissed pmattmann’s stale review June 24, 2021 07:36

Gemäss @BacLuc, bitte restliche Commits noch reviewen

@carlobeltrame carlobeltrame merged commit 9a45f07 into ecamp:devel Jul 14, 2021
@BacLuc
Copy link
Contributor Author

BacLuc commented Jul 17, 2021

Ok, ging es doch ohne rebase?

@BacLuc BacLuc deleted the frontend-acl branch July 17, 2021 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pmattmann pmattmann pmattmann approved these changes

@carlobeltrame carlobeltrame carlobeltrame approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BacLuc @carlobeltrame @pmattmann