Skip to content

INVOICES: 360° Sicht #205

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

Closed
6 tasks done
mlue00 opened this issue Aug 5, 2022 · 3 comments · Fixed by hitobito/hitobito#1846
Closed
6 tasks done

INVOICES: 360° Sicht #205

mlue00 opened this issue Aug 5, 2022 · 3 comments · Fixed by hitobito/hitobito#1846

Comments

@mlue00
Copy link

mlue00 commented Aug 5, 2022

Sichtbarkeit von Tab Rechnungen.

Personen mit :finance-Ability sehen das Tab bei Personen im ganzen Layer (ohne Layer unterhalb)
Auf der eigenen Person soll dieses Tab immer sichtbar sein.

https://help.puzzle.ch/#ticket/zoom/4338 beschreibt das Problem noch mit konkreten Personennamen.

Der Fehler scheint im Zusammenhang mit hitobito/hitobito#474 entstanden zu sein. Hier scheint die Ursache zu sein:

https://github.com/hitobito/hitobito/blob/da6bb02636fc203628b320c221549dcf50e75302/app/helpers/sheet/person.rb#L26

Es könnte sein, dass hier statt person.finance_groups.present? eher current_user.finance_groups.present? gemeint ist. Dieser check sollte aber ohnehin in die InvoiceAbility.

Tech-Spec

  • Verschickte Rechnungen können auf einer Person angezeigt werden.
  • Es werden Rechnungen der entsprechenden Ebene angezeigt.
  • Personen mit Finanzberechtigungen sehen die Rechnungen.
  • Man sieht seine eigenen Rechnungen

Fehlerbeschreibung

Referenzen

ToDo

  • InvoiceAbility anpassen, um die Berechtigung nur dort zu kapseln
  • Sichtbarkeit vom Tab "invoices" anpassen
  • Specs schreiben
  • Mit angemessener Rolle "durchklicken"
  • Mit Root-User durchklicken, um erneuten Bug auszuschliessen
  • CHANGELOG-Eintrag unter "unreleased" unten hinzufügen
@mlue00 mlue00 changed the title MODULE: Neues Feature INVOICES: 360° Sicht Aug 5, 2022
@carlobeltrame
Copy link
Member

carlobeltrame commented Aug 6, 2022

Auf jeder Person? Ist das nicht dasselbe wie hitobito/hitobito#1803?

@olibrian
Copy link
Member

Wurde bereits im hitobito/hitobito#1803 und hitobito/hitobito#1776 ausführlich beschrieben. Mit dem Abschluss von diesem Ticket sollten die beiden anderen obsolet werden.

@richardjubla
Copy link

Hinweis zum jubla-Wagon oder Instanzen welche das Rechnungs-Modul nicht verwenden. Der Tab Rechnungen auf der Person kann schon immer sichtbar sein, WENN das Rechnungs-Modul verwendet wird.
Bei uns ist der Tab Rechnungen aktiv, obwohl wir das Rechnungs-Modul nicht verwenden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants