Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
K
kohinos-tav
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • agplv3
  • kohinos-tav
  • Merge Requests
  • !106

Project 'cooperatic/kohinos-tav' was moved to 'agplv3/kohinos-tav'. Please update any links and bookmarks that may still have the old path.
Merged
Opened Jun 10, 2024 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

allow tresorier to flux admin

  • Discussion 3
  • Commits 2
  • Changes 6
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 13, 2024
    Last updated by Damien Moulard Jun 13, 2024
    src/Migrations/Version20240610105532.php 0 → 100644
    1 <?php
    • Yvon Kerdoncuff @Yvon commented Jun 13, 2024
      Master

      Je vois plusieurs problèmes avec cette solution.

      1. 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.

      2. 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.

      3. 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.

      Je vois plusieurs problèmes avec cette solution. 1. 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 7142d163e495ca647dd27801fde27a27f440b10b pour ajouter le droit ROLE_ADMIN_ADHERENT_GERER_LIST au trésorier. 2. Un autre problème, qui ne se posait peut-être pas dans le cas du 7142d163e495ca647dd27801fde27a27f440b10b (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. 3. 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.
    • Damien Moulard @DamienM commented Jun 13, 2024
      Master

      discuté en direct, point 1 ok, point 2 & 3 semble acceptable pour le menu Flux mais à communiquer

      Edited Jun 13, 2024 by Damien Moulard
      discuté en direct, point 1 ok, point 2 & 3 semble acceptable pour le menu Flux mais à communiquer
    • Damien Moulard @DamienM commented Jun 13, 2024
      Master

      corrigé

      corrigé
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • bf30cabf - add tresorier rights to access Adherent (list only) & Flux menus in fixtures

    Compare with previous version

    Jun 13, 2024

    added 1 commit

    • bf30cabf - add tresorier rights to access Adherent (list only) & Flux menus in fixtures

    Compare with previous version

    added 1 commit <ul><li>bf30cabf - add tresorier rights to access Adherent (list only) &amp; Flux menus in fixtures</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/106/diffs?diff_id=3151&start_sha=b421c554ef8401478e160944ea24531e68dbc843)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    Jun 13, 2024

    merged

    merged
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit fbf72e45

    Jun 13, 2024

    mentioned in commit fbf72e45

    mentioned in commit fbf72e455ae447b26a3efdf3504295027ecdf7cf
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: agplv3/kohinos-tav!106
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.