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
  • !84

Merged
Opened Apr 17, 2024 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

add fields to fix balance in fiche adherent form for super admin only

  • Discussion 8
  • Commits 7
  • Changes 20
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Apr 19, 2024
    Resolved by Damien Moulard Apr 25, 2024
    src/Admin/AdherentAdmin.php
    387 421 if ($adherent->getCotisationAmount() < $minCotisationAmount) {
    388 422 $event->getForm()->get('cotisationAmount')->addError(new FormError('Le montant minimum est de ' . $minCotisationAmount . '€ (selon les données du foyer indiquées)'));
    389 423 }
    424
    425 // try to fix balance if required
    426 if($this->security->isGranted('ROLE_SUPER_ADMIN')
    427 && $event->getForm()->has('fixedBalance')
    428 && $event->getForm()->get('fixedBalance')->getData()
    429 && $event->getForm()->get('fixedBalance')->getData() >= 0) {
    430 $password = "cestl@fete";
    • Damien Moulard @DamienM commented Apr 19, 2024
      Master

      je mettrais le mot de passe en paramètre d'environnement : plus simple à modifier, plus maléable pour les différentes instances

      je mettrais le mot de passe en paramètre d'environnement : plus simple à modifier, plus maléable pour les différentes instances
    • Yvon Kerdoncuff @Yvon commented Apr 22, 2024
      Master

      ok

      ok
    • Yvon Kerdoncuff @Yvon

      changed this line in version 2 of the diff

      Apr 22, 2024

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/84/diffs?diff_id=2950&start_sha=bf498140370d7f3129d867d8c1c419bfe808b758#ef2be88c52e44d777fb4d25b33863ee6be1a97f5_430_433)
      Toggle commit list
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion Apr 19, 2024
    Last updated by Yvon Kerdoncuff Apr 22, 2024
    • Damien Moulard @DamienM commented Apr 19, 2024
      Master

      Je peux comprendre que tu n'aies pas voulu faire de popup, je pense néanmoins que ça serait bien de au moins cacher les champs par défaut, et de les afficher au clic sur un bouton. Ça demande juste un peu de js donc pas trop demandant.

      Je pense que c'est bien de le faire car c'est une opération exceptionnelle et sensible, qui donc je pense ne devrait pas apparaitre au même "niveau" que le reste.

      Je peux comprendre que tu n'aies pas voulu faire de popup, je pense néanmoins que ça serait bien de au moins cacher les champs par défaut, et de les afficher au clic sur un bouton. Ça demande juste un peu de js donc pas trop demandant. Je pense que c'est bien de le faire car c'est une opération exceptionnelle et sensible, qui donc je pense ne devrait pas apparaitre au même "niveau" que le reste.
    • Yvon Kerdoncuff @Yvon commented Apr 22, 2024
      Master

      ok, fait mais je suis pas très à l'aise avec ce genre de manip js j'ai un peu galéré

      ok, fait mais je suis pas très à l'aise avec ce genre de manip js j'ai un peu galéré
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 3 commits

    • 72f78dda - put fix balance adherent password in .env file
    • 9a09f9e7 - hide fix balance fields by default and show them on click on a btn
    • 8642988e - yarn

    Compare with previous version

    Apr 22, 2024

    added 3 commits

    • 72f78dda - put fix balance adherent password in .env file
    • 9a09f9e7 - hide fix balance fields by default and show them on click on a btn
    • 8642988e - yarn

    Compare with previous version

    added 3 commits <ul><li>72f78dda - put fix balance adherent password in .env file</li><li>9a09f9e7 - hide fix balance fields by default and show them on click on a btn</li><li>8642988e - yarn</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/84/diffs?diff_id=2950&start_sha=bf498140370d7f3129d867d8c1c419bfe808b758)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    resolved all discussions

    Apr 22, 2024

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    changed target branch from ssa-gironde to develop

    Apr 22, 2024

    changed target branch from ssa-gironde to develop

    changed target branch from `ssa-gironde` to `develop`
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 1cd82b94 - 6213 : send mail on ceiling cut

    Compare with previous version

    Apr 22, 2024

    added 1 commit

    • 1cd82b94 - 6213 : send mail on ceiling cut

    Compare with previous version

    added 1 commit <ul><li>1cd82b94 - 6213 : send mail on ceiling cut</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/84/diffs?diff_id=2952&start_sha=8642988e2a91f16d16a18cb932f19998151c5676)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 9a636fb9 - 6212 : create new flux types to send email after super admin balance fix

    Compare with previous version

    Apr 24, 2024

    added 1 commit

    • 9a636fb9 - 6212 : create new flux types to send email after super admin balance fix

    Compare with previous version

    added 1 commit <ul><li>9a636fb9 - 6212 : create new flux types to send email after super admin balance fix</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/84/diffs?diff_id=2974&start_sha=1cd82b94e63808632e887b36fefe037a3aceb2d7)
    Toggle commit list
  • Damien Moulard @DamienM

    resolved all discussions

    Apr 25, 2024

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard
    @DamienM started a discussion Apr 25, 2024
    Last updated by Yvon Kerdoncuff Apr 25, 2024
    • Damien Moulard @DamienM commented Apr 25, 2024
      Master

      Le fichier admin js résultant du build n'est plus dans le suivi de version (tout le js côté admin ne fonctionne plus du coup).

      On peut voir dans les différences de fichiers que l'ancien build est bien supprimé, le nouveau fichier de build à récupérer est bien indiqué dans le manifest, mais le fichier de build en lui-même n'est pas dans le commit.

      Edited Apr 25, 2024 by Damien Moulard
      Le fichier admin js résultant du build n'est plus dans le suivi de version (tout le js côté admin ne fonctionne plus du coup). On peut voir dans les différences de fichiers que l'ancien build est bien supprimé, le nouveau fichier de build à récupérer est bien indiqué dans le manifest, mais le fichier de build en lui-même n'est pas dans le commit.
    • Damien Moulard @DamienM commented Apr 25, 2024
      Master

      Quand je modifie du JS, je lance simplement yarn run encore prod avant de commit et je ne touche à rien d'autre (je sais pas si ça t'aide)

      Quand je modifie du JS, je lance simplement `yarn run encore prod` avant de commit et je ne touche à rien d'autre (je sais pas si ça t'aide)
    • Yvon Kerdoncuff @Yvon commented Apr 25, 2024
      Master

      j'avais oublié de commit ces fichiers, c'est fait

      j'avais oublié de commit ces fichiers, c'est fait
    Please register or sign in to reply
  • Damien Moulard @DamienM commented Apr 25, 2024
    Master

    fonctionnalité ok, test à faire en pp pour l'envoie de mail

    fonctionnalité ok, test à faire en pp pour l'envoie de mail
  • Damien Moulard @DamienM

    closed

    Apr 25, 2024

    closed

    closed
    Toggle commit list
  • Damien Moulard @DamienM

    reopened

    Apr 25, 2024

    reopened

    reopened
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 9639b5c6 - add missing public/build/admin...js

    Compare with previous version

    Apr 25, 2024

    added 1 commit

    • 9639b5c6 - add missing public/build/admin...js

    Compare with previous version

    added 1 commit <ul><li>9639b5c6 - add missing public/build/admin...js</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/84/diffs?diff_id=2980&start_sha=9a636fb90a99cd8e42bac5f4f3d5f4da52761f48)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 7a12ca10

    Apr 26, 2024

    mentioned in commit 7a12ca10

    mentioned in commit 7a12ca1043aadea5273ab73e8f80da6b971c2cc9
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    Apr 26, 2024

    merged

    merged
    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!84
×

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.