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

Implement/check Changelog permissions #1478

Open
6 of 7 tasks
daniel-illi opened this issue Jan 6, 2025 · 0 comments · May be fixed by #1481 or hitobito/hitobito#3117
Open
6 of 7 tasks

Implement/check Changelog permissions #1478

daniel-illi opened this issue Jan 6, 2025 · 0 comments · May be fixed by #1481 or hitobito/hitobito#3117

Comments

@daniel-illi
Copy link
Contributor

daniel-illi commented Jan 6, 2025

Im SAC Wagon soll der Zugriff auf das Changelog sowie die Links auf die Commits der Wagons eingeschränkt werden.
Dafür soll im Core eine permission auf ChangelogEntry eingeführt werden, welche allen usern darauf :index erlaubt. Im footer view partial sowie im ChangelogController soll die Berechtigung geprüft werden.
Im SAC Wagon soll die permission überschrieben werden, so dass :index nur erlaubt ist für user mit Rolle in der Geschäftsstelle Gruppe.

Tech-Spec / ToDo

  • im Core:

    • VariousAbility erweitern:
      on(ChangelogEntry) do
        class_side(:index).all
      end
    • view partial app/views/layouts/_footer.html.haml anpassen:
      Version Link und #detail-info nur anzeige, falls can?(:index, ChangelogEntry),
      andernfalls nur Version unverlinkt anzeigen.
  • im SAC wagon die VariousAbility erweitern:

    on(ChangelogEntry) do
      class_side(:index).if_mitarbeiter_geschaeftsstelle
    end
  • Specs schreiben

  • Mit angemessener Rolle "durchklicken"

  • CHANGELOG-Eintrag unter "unreleased" unten hinzufügen

@daniel-illi daniel-illi self-assigned this Jan 6, 2025
@daniel-illi daniel-illi transferred this issue from hitobito/hitobito Jan 6, 2025
@daniel-illi daniel-illi removed their assignment Jan 6, 2025
@daniel-illi daniel-illi changed the title Implement ChangelogAbility Implement/check Changelog permissions Jan 6, 2025
@njaeggi njaeggi self-assigned this Jan 7, 2025
@njaeggi njaeggi removed their assignment Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants