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

Merged
Opened Jan 04, 2023 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

3576 inscription modification habitant

  • Discussion 3
  • Commits 5
  • Changes 12
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Jan 05, 2023
    Resolved by Damien Moulard Jan 05, 2023
    src/Admin/AdherentAdmin.php
    189 191 }
    190 192 ])
    193 ->add('moyenDePaiement', ChoiceType::class, [
    194 'required' => true,
    195 'label' => 'Moyen de paiement :',
    196 'choices' => [
    197 "CB" => "CB",
    198 "espèces" => "espèces",
    199 "chèque" => "chèque",
    200 "prélèvement" => "prélèvement"
    201 ],
    202 'empty_data' => null,
    203 'placeholder' => 'Choisir un moyen de paiement'
    204 ])
    205 ->add('jourPrelevement', ChoiceType::class, [
    206 'required' => true,
    • Damien Moulard @DamienM commented Jan 05, 2023
      Master

      ce champ ne doit pas être required, car le champ n'est pas forcément affiché/à remplir

      ce champ ne doit pas être required, car le champ n'est pas forcément affiché/à remplir
    • Yvon Kerdoncuff @Yvon

      changed this line in version 2 of the diff

      Jan 05, 2023

      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/23/diffs?diff_id=1992&start_sha=077b3610dbfe6b14cf76f0c2f97d529655f3e172#ef2be88c52e44d777fb4d25b33863ee6be1a97f5_206_206)
      Toggle commit list
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Jan 05, 2023
    Resolved by Damien Moulard Jan 05, 2023
    src/Admin/AdherentAdmin.php
    202 'empty_data' => null,
    203 'placeholder' => 'Choisir un moyen de paiement'
    204 ])
    205 ->add('jourPrelevement', ChoiceType::class, [
    206 'required' => true,
    207 'label' => 'Jour de prélèvement :',
    208 'choices' => $this->daysOfMonth(),
    209 'empty_data' => null,
    210 'placeholder' => 'Choisir un jour de prélèvement'
    211 ])
    212 ->add('mailRappelCotisation', CheckboxType::class, [
    213 'required' => false,
    214 'label' => 'Recevoir un rappel du paiement de ma cotisation par mail',
    215 ])
    216 ->add('jourMailRappelCotisation', ChoiceType::class, [
    217 'required' => true,
    • Damien Moulard @DamienM commented Jan 05, 2023
      Master

      idem

      idem
    • Yvon Kerdoncuff @Yvon

      changed this line in version 2 of the diff

      Jan 05, 2023

      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/23/diffs?diff_id=1992&start_sha=077b3610dbfe6b14cf76f0c2f97d529655f3e172#ef2be88c52e44d777fb4d25b33863ee6be1a97f5_217_217)
      Toggle commit list
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 6380045e - fix required attribute

    Compare with previous version

    Jan 05, 2023

    added 1 commit

    • 6380045e - fix required attribute

    Compare with previous version

    added 1 commit <ul><li>6380045e - fix required attribute</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/23/diffs?diff_id=1992&start_sha=077b3610dbfe6b14cf76f0c2f97d529655f3e172)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon commented Jan 05, 2023
    Master

    Corrections pushées !

    Corrections pushées !
  • Damien Moulard @DamienM

    resolved all discussions

    Jan 05, 2023

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    Jan 05, 2023

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 368bf173

    Jan 05, 2023

    mentioned in commit 368bf173

    mentioned in commit 368bf17397be26fc6e6b4bd15d2cf4a98ae982da
    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!23
×

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.