allow tresorier to flux admin
-
src/Migrations/Version20240610105532.php 0 → 100644
1 <?php -
Master
Je vois plusieurs problèmes avec cette solution.
Elle ne sera pas prise en compte, par exemple dans le cas d'une réinstallation du projet SSA-G (pour rappel, dans la procédure d'installation, après l'execution des migrations, les modifications de CONTENU de la bdd sont effacées avant de jouer un fichier de fixtures). Le fix est facile, il suffit d'ajouter ROLE_ADMIN_FLUX_GERER_ALL au fichier de fixtures fixtures-ssagironde-pro.yaml dans la section __construct Trésorier de usergroup_tresorier, comme cela a été fait dans le commit 7142d163 pour ajouter le droit ROLE_ADMIN_ADHERENT_GERER_LIST au trésorier.
Un autre problème, qui ne se posait peut-être pas dans le cas du 7142d163 (parce qu'il s'agissait de donner accès à une fonctionnalité configurable), est que lorsque le code sera mergé et mis en prod, TAV va automatiquement recevoir cette augmentation des droits du trésorier. C'est la raison pour laquelle je leur ai envoyé un mail pour leur demander si la modification leur convient. Si c'est OK pour eux alors cela résoud ce second problème, sinon il va falloir imaginer quelque chose.
Enfin, dans la même veine que 2 se pose le problème de la mise à jour des instances des monnaies locales qui utilisent le projet kohinos, dans l'hypothèse d'une future convergence des codes. Et là on a un os car la modification sera automatiquement appliquée sur ces instances lorsque seront jouées les migrations. Pour ce troisième problème, je serais tenté de botter en touche en explicitant le problème dans INSTALL.md et de repousser la recherche d'une solution à plus tard.
-
Master
discuté en direct, point 1 ok, point 2 & 3 semble acceptable pour le menu Flux mais à communiquer
-
Master
corrigé
-
-
added 1 commit
- bf30cabf - add tresorier rights to access Adherent (list only) & Flux menus in fixtures
Toggle commit list -
merged
Toggle commit list