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

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

prevent cancelling transactions which happened before last reconversion

  • Discussion 4
  • Commits 2
  • Changes 1
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff May 02, 2024
    Last updated by Yvon Kerdoncuff May 06, 2024
    src/Controller/UserController.php
    285 287 return $this->redirectToRoute('index');
    286 288 }
    287 289
    290 //Prevent cancelling transactions which happened before the most recent reconversion (SSA Gironde sprint 4 ; see sprint 5 for CCAS specific case)
    • Damien Moulard @DamienM commented May 02, 2024
      Master

      (SSA Gironde sprint 4 ; see sprint 5 for CCAS specific case) -> (to avoid refunding money that's been reconverted)

      Comme ça le code est indépendant du système de suivi de tickets utilisé.

      `(SSA Gironde sprint 4 ; see sprint 5 for CCAS specific case)` -> `(to avoid refunding money that's been reconverted)` Comme ça le code est indépendant du système de suivi de tickets utilisé.
    • Damien Moulard @DamienM commented May 02, 2024
      Master

      D'ailleurs, before the most recent reconversion -> before a reconversion (c'est techniquement ce qu'il se passe dans la vérification)

      D'ailleurs, `before the most recent reconversion` -> `before a reconversion` (c'est techniquement ce qu'il se passe dans la vérification)
    • Yvon Kerdoncuff @Yvon

      changed this line in version 2 of the diff

      May 06, 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/87/diffs?diff_id=2997&start_sha=970e19ced8fa44d035c9fd37b95237a63c3280b7#87b99206717ccce222f05190f36b6273e3ea6228_290_290)
      Toggle commit list
    • Yvon Kerdoncuff @Yvon commented May 06, 2024
      Master

      Remarque 1 : d'accord avec toi. J'utilise aussi les commentaires pour te passer des infos mais c'est peut-être pas une bonne idée car je constate que tu me demandes parfois des reprises pour que le commentaire soit plus adapté à une personnne extérieure. Je voulais te signaler ici que j'avais découpé en deux la demande de blocage des annulations antérieures à la dernière reconversion en deux tickets : la partie non CCAS et la partie CCAS qui sera traitée ultérieurement.

      Remarque 2 : je la trouve non pertinente car les deux formulations concurrentes sont équivalentes, mais du coup je préfère faire les modifs que tu recommandes plutôt que de différer le merge.

      Remarque 1 : d'accord avec toi. J'utilise aussi les commentaires pour te passer des infos mais c'est peut-être pas une bonne idée car je constate que tu me demandes parfois des reprises pour que le commentaire soit plus adapté à une personnne extérieure. Je voulais te signaler ici que j'avais découpé en deux la demande de blocage des annulations antérieures à la dernière reconversion en deux tickets : la partie non CCAS et la partie CCAS qui sera traitée ultérieurement. Remarque 2 : je la trouve non pertinente car les deux formulations concurrentes sont équivalentes, mais du coup je préfère faire les modifs que tu recommandes plutôt que de différer le merge.
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion May 02, 2024
    • Damien Moulard @DamienM commented May 02, 2024
      Master

      Ok pour merge après changement du commentaire

      Ok pour merge après changement du commentaire
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 62776c66 - change comment

    Compare with previous version

    May 06, 2024

    added 1 commit

    • 62776c66 - change comment

    Compare with previous version

    added 1 commit <ul><li>62776c66 - change comment</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/87/diffs?diff_id=2997&start_sha=970e19ced8fa44d035c9fd37b95237a63c3280b7)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    May 06, 2024

    merged

    merged
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 9c6a71e3

    May 06, 2024

    mentioned in commit 9c6a71e3

    mentioned in commit 9c6a71e3d2f83d25fde40503ee42562ee229eb4b
    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!87
×

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.