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

Merged
Opened Feb 20, 2024 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

add presta conventionnement percentage

  • Discussion 5
  • Commits 3
  • Changes 4
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion Feb 21, 2024
    Resolved by Damien Moulard Feb 21, 2024
    • Damien Moulard @DamienM commented Feb 21, 2024
      Master

      Par soucis de cohérence visuelle j'aurais placé le select de Conventionnement en dessous du Taux de reconversion

      image

      Par soucis de cohérence visuelle j'aurais placé le select de Conventionnement en dessous du Taux de reconversion ![image](/uploads/946ddb671c3202bfea7f448d608dc60f/image.png)
    • Yvon Kerdoncuff @Yvon commented Feb 21, 2024
      Master

      Dans le flow, le gestionnaire va devoir cocher la case activé et saisir le conventionnement, du coup je trouvais ça pas mal qu'ils soient à côté (ça fait aussi un peu moins de code).

      Dans le flow, le gestionnaire va devoir cocher la case activé et saisir le conventionnement, du coup je trouvais ça pas mal qu'ils soient à côté (ça fait aussi un peu moins de code).
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion Feb 21, 2024
    Resolved by Damien Moulard Feb 21, 2024
    • Damien Moulard @DamienM commented Feb 21, 2024
      Master

      On ne peut pas mettre une valeur nulle. Si je ne renseigne aucune valeur j'ai l'erreur :

      Expected argument of type "float", "null" given at property path "conventionnement".

      On ne peut pas mettre une valeur nulle. Si je ne renseigne aucune valeur j'ai l'erreur : `Expected argument of type "float", "null" given at property path "conventionnement".`
    • Yvon Kerdoncuff @Yvon commented Feb 21, 2024
      Master

      Effectivement, saisir une valeur ne devrait être obligatoire que si le point de vente est activé (car ensuite ce taux va être utilisé à l'encaissement pour calculer la part du montant payable en mona, en tous cas c'est ce que je crois commencer à comprendre).

      Effectivement, saisir une valeur ne devrait être obligatoire que si le point de vente est activé (car ensuite ce taux va être utilisé à l'encaissement pour calculer la part du montant payable en mona, en tous cas c'est ce que je crois commencer à comprendre).
    • Yvon Kerdoncuff @Yvon commented Feb 21, 2024
      Master

      Mais c'est un peu plus compliqué donc je me suis contenté de permettre de saisir une valeur nulle.

      Mais c'est un peu plus compliqué donc je me suis contenté de permettre de saisir une valeur nulle.
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • ac341f87 - allow conventionnement to be null

    Compare with previous version

    Feb 21, 2024

    added 1 commit

    • ac341f87 - allow conventionnement to be null

    Compare with previous version

    added 1 commit <ul><li>ac341f87 - allow conventionnement to be null</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/53/diffs?diff_id=2777&start_sha=3d14875f811f208f7b7c887108db4eb93d1899c1)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • c8f68538 - remove useless migration

    Compare with previous version

    Feb 21, 2024

    added 1 commit

    • c8f68538 - remove useless migration

    Compare with previous version

    added 1 commit <ul><li>c8f68538 - remove useless migration</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/53/diffs?diff_id=2778&start_sha=ac341f872431b7b80ac5a956bdac0de05a727ee2)
    Toggle commit list
  • Damien Moulard @DamienM

    resolved all discussions

    Feb 21, 2024

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    Feb 21, 2024

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 20d36a21

    Feb 21, 2024

    mentioned in commit 20d36a21

    mentioned in commit 20d36a2110fbe3115ea22bf5f385a5ab410fc9ec
    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!53
×

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.