Přidán přístup k plabám pro hospodáře a vedoucí VzA #2619
Conversation
marekdedic
changed the title
| @@ -236,7 +236,7 @@ private function assertCanViewBankAccount(int $id): void | |||
| if ( | |||
| $role->getUnitId() === $account->getUnitId() | |||
| && $role->isBasicUnit() | |||
| && ($role->isAccountant() || $role->isOfficer() || $role->isEventManager()) | |||
| && ($role->isAccountant() || $role->isOfficer() || $role->isEventManager() || $role->isEducationAccountant()) | |||
There was a problem hiding this comment.
a proc to nepripojit pod funkci role->isAccountant() a dělat na to samostatnou?
A nenastane v obou případech, že hospodář VzA dostane přístup k uctu i jednotky jako takové?
There was a problem hiding this comment.
Jo, určitě by to šlo připojit pod isAccountant(), ale přišlo mi lepší nemíchat 2 různé role do jedné funkce, zvlášť když ta vnitřní logika těch rolí je úplně jiná.
Ano, hospodář VzA tím dostane přístup i k platbám jednotky. Mně to přijde asi v pohodě, z důvodů popsaných v těle PR.
There was a problem hiding this comment.
Za mě je to stále Accountant, tak bych to spíš spojil. Protože stejně jsme udělali Officer pres více úrovní a rolí.
S tím přístupem jsem si toho nejdřív nevšiml. Každopádně bych přidal i hlášku do plateb, že přístup k platbám mají i role VzA pod danou jednotkou.
There was a problem hiding this comment.
Ok, sloučil jsem ty funkce. Co myslíš hláškou? Permanentní baner mi přijde jako overkill...
There was a problem hiding this comment.
Jako hospodar, co si zadá ucet do h.skautingu neocekávám, že se k tomu automaticky dostane i vedoucí vzdělávačky. Pokud existuje nějaký precedens pro přístup lidí z VzA k jednotce, tak to udělat podle toho @manulekonecna
There was a problem hiding this comment.
Tak jako precedens je podle mě trochu v cesťácích - tam taky s rolí vzdělávačky vidím i cesťáky a auta střediska...
Napadá mě ještě tam ten baner dát a třeba za 3 měsíce ho zase zrušit. Klidně na to připravím druhé PR...
There was a problem hiding this comment.
jeste bych rád pohled @manulekonecna nebo @jerrysohn
There was a problem hiding this comment.
Jen ještě pro kontext doplním, jaké je typické řešení tohohle problému ze strany pořádajících jednotek - hospodář dostane v ISu roli správce akcí, potažmo vedoucí/admin... :D
marekdedic
force-pushed
the
education-economist-payment-access
branch
from
7e6004d
to
b81a64b
Compare
|
@sinacek @jerrysohn @manulekonecna Ahoj, potřeboval bych nějaké vyjádření k tomuto půl roku starému PR - otevírám druhý ročník naší VzA, ve kterém mě tohle blokuje. Stejný problém pak v zápětí budu řešit i pro cesťáky, kde bych jako hospodář VzA potřeboval taky větší přístup - prostě teď některé důležité části h.skautingu jsou nepoužitelné bez role u jednotky, kterou podle mě nemám mít. Díky. |
marekdedic
force-pushed
the
education-economist-payment-access
branch
from
b81a64b
to
3d5d811
Compare
S tímhle PR mají hospodáři VzA přístup ke všem platbám dané jednotky - nejsou tam víc granulární oprávnění, ale mně to asi přijde v pohodě, hospodáři VzA jsou typicky lidi, které ta jednotka nějakým způsobem stejně zná a hospodář střediska s nimi nutně interaguje...
Stejně tak jsem rovnou přidal vedoucí VzA, ty dokonce schvaluje střediskovka...
Closes #2617