Skip to content

T
third-party
Opened by  

3867-ratio-button-binome-option

{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Félicie @felicie

    added 5 commits

    • f54ca15d...04f149b4 - 4 commits from branch dev_cooperatic
    • 290dc9f7 - Merge branch 'dev_cooperatic' into 3867-ratio-button-binome-option

    Compare with previous version

    added 5 commits

    • f54ca15d...04f149b4 - 4 commits from branch dev_cooperatic
    • 290dc9f7 - Merge branch 'dev_cooperatic' into 3867-ratio-button-binome-option

    Compare with previous version

    Toggle commit list
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff
    Resolved by Damien Moulard
    members/static/js/inscriptions.js
    279 296 ($(active_asso_area[0]).attr('id') === "existing_member_choice" && $('#existing_member_choice_action .chosen_associate div.member').length > 0)
    280 297 ) {
    281 298 associated_data_ok = true;
    299 } else if ($(active_asso_area[0]).attr('id') === "") {
    300 associated_data_ok = false;
    301 errors.push("Un des deux choix doit être sélectionné");
    282 302 }
    283 303 if (associated_data_ok === false) errors.push("Le membre 'titulaire' du binôme n'est pas défini");
    284 304 }
    285 305
    306 console.log($(active_asso_area[0]).attr('id'))
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on the diff
    Resolved by Félicie
    members/static/js/inscriptions.js
    279 296 ($(active_asso_area[0]).attr('id') === "existing_member_choice" && $('#existing_member_choice_action .chosen_associate div.member').length > 0)
    280 297 ) {
    281 298 associated_data_ok = true;
    299 } else if ($(active_asso_area[0]).attr('id') === "") {
    300 associated_data_ok = false;
    301 errors.push("Un des deux choix doit être sélectionné");
    • Damien Moulard @DamienM commented
      Developer

      Ce message est trop vague. Peut-être "Un des deux choix concernant la mise en binôme doit être sélectionné"?

      Edited
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff
    Resolved by Damien Moulard
    members/static/js/inscriptions.js
    268 270 }
    269 271 }
    270 272
    273 if (associated_area_actived.show()) {
    • Damien Moulard @DamienM commented
      Developer

      Cette ligne ne va pas. Tu lances une action dans le if. Ça a pour effet que si je met Non puis clique sur Valider, ça affiche le formulaire (et inversement). Ici, il faut juste vérifier si le formulaire est affiché (ce qui était fait avant ligne 615 : if ($('#associate_area').is(':visible')))

    • Félicie @felicie commented
      Developer

      Ah oui, je vois ! Je corrige de suite !

    • Félicie @felicie

      changed this line in version 3 of the diff

      changed this line in version 3 of the diff

      Toggle commit list
    Please register or sign in to reply
  • Félicie @felicie

    added 2 commits

    Compare with previous version

    added 2 commits

    Compare with previous version

    Toggle commit list
  • Félicie @felicie

    resolved all discussions

    resolved all discussions

    Toggle commit list
  • Damien Moulard @DamienM

    merged

    merged

    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 71e4b8bb

    mentioned in commit 71e4b8bb

    Toggle commit list
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 to comment