Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
T
third-party
  • 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
  • cooperatic-foodcoops
  • third-party
  • Merge Requests
  • !14

Merged
Opened May 14, 2021 by Colin MORAND@colin 
  • Report abuse
Report abuse

ticket_819 Changer le texte du bouton de confirmation de présence à un service

modifié : members/views.py modifié : templates/members/index.html modifié : coops_configurations/config_lacagette.py

Edited May 14, 2021 by Colin MORAND
  • Discussion 8
  • Commits 2
  • Pipelines 2
  • Changes 3
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Colin MORAND @colin

    changed the description

    May 14, 2021

    changed the description

    changed the description
    Toggle commit list
  • Colin MORAND @colin

    assigned to @fracolo

    May 14, 2021

    assigned to @fracolo

    assigned to @fracolo
    Toggle commit list
  • Paul
    @paul_lgds started a discussion on an old version of the diff May 14, 2021
    Last updated by Colin MORAND May 16, 2021
    members/views.py
    23 23 'WELCOME_ENTRANCE_MSG': getattr(settings, 'WELCOME_ENTRANCE_MSG', 'Bienvenue !'),
    24 24 'WELCOME_SUBTITLE_ENTRANCE_MSG': getattr(settings, 'WELCOME_SUBTITLE_ENTRANCE_MSG', ''),
    25 25 'ENTRANCE_SHOPPING_BTN': getattr(settings, 'ENTRANCE_SHOPPING_BTN', 'Je viens faire mes courses'),
    26 'ENTRANCE_SERVICE_BTN': getattr(settings, 'ENTRANCE_SERVICE_BTN', 'Je viens faire mon service')
    26 'ENTRANCE_SERVICE_BTN': getattr(settings, 'ENTRANCE_SERVICE_BTN', 'Je viens faire mon service'),
    27 'BUTTON_CONFIRME_PRESENT' : getattr(settings, 'BUTTON_CONFIRME_PRESENT', '')
    • Paul @paul_lgds commented May 14, 2021

      Est-ce qu'on ne garderait pas la valeur précédente comme valeur par défaut pour cette variable de config. On sera sûr de ne pas « casser » l'affichage pour les configs des autres coops :)

      -         'BUTTON_CONFIRME_PRESENT' : getattr(settings, 'BUTTON_CONFIRME_PRESENT', '')
      +         'BUTTON_CONFIRME_PRESENT' : getattr(settings, 'BUTTON_CONFIRME_PRESENT', 'Présent.e')
      Est-ce qu'on ne garderait pas la valeur précédente comme valeur par défaut pour cette variable de config. On sera sûr de ne pas « casser » l'affichage pour les configs des autres coops :) ```diff - 'BUTTON_CONFIRME_PRESENT' : getattr(settings, 'BUTTON_CONFIRME_PRESENT', '') + 'BUTTON_CONFIRME_PRESENT' : getattr(settings, 'BUTTON_CONFIRME_PRESENT', 'Présent.e') ```
    • Colin MORAND @colin

      changed this line in version 2 of the diff

      May 16, 2021

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/14/diffs?diff_id=623&start_sha=eb2a0f69a23ea46e1cc837f2310c3e1654eb6ec9#6ee3779c4db33f6dd20fd893ea7bbde91722982b_27_27)
      Toggle commit list
    Please register or sign in to reply
  • Colin MORAND @colin

    added 1 commit

    • 8b282494 - modifié : coops_configurations/config_lacagette.py

    Compare with previous version

    May 16, 2021

    added 1 commit

    • 8b282494 - modifié : coops_configurations/config_lacagette.py

    Compare with previous version

    added 1 commit <ul><li>8b282494 - modifié : coops_configurations/config_lacagette.py</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/14/diffs?diff_id=623&start_sha=eb2a0f69a23ea46e1cc837f2310c3e1654eb6ec9)
    Toggle commit list
  • Colin MORAND @colin commented May 16, 2021
    Developer

    Bonjour, J'ai fait la modification pour avoir une valeur par default. J'ai aussi changer le nom de la variable en mettant BTN a la fin comme la variable ENTRANCE_SERVICE_BTN et `ENTRANCE_SHOPPING_BTN'. Cdt, Colin

    Bonjour, J'ai fait la modification pour avoir une valeur par default. J'ai aussi changer le nom de la variable en mettant BTN a la fin comme la variable `ENTRANCE_SERVICE_BTN` et `ENTRANCE_SHOPPING_BTN'. Cdt, Colin
  • Paul @paul_lgds commented May 17, 2021

    Merci @colin.

    À terme je pense qu'il serait préférable d'utiliser des fichiers spécifiques qui contiennent seulement les textes (comme des fichiers de traductions, au format YAML ou JSON par exemple). Ça permettrait de toujours utiliser des clés pour les textes dans le code, associés à des valeurs dans les « fichiers de traductions » qui pourraient donc être différents pour chaque coop.

    Merci @colin. À terme je pense qu'il serait préférable d'utiliser des fichiers spécifiques qui contiennent seulement les textes (comme des fichiers de traductions, au format YAML ou JSON par exemple). Ça permettrait de toujours utiliser des clés pour les textes dans le code, associés à des valeurs dans les « fichiers de traductions » qui pourraient donc être différents pour chaque coop.
  • Colin MORAND @colin commented May 18, 2021
    Developer

    Bonjour @paul_lgds, Qui doit normalement fermer le merge request ?

    Bonjour @paul_lgds, Qui doit normalement fermer le merge request ?
  • Paul @paul_lgds commented May 18, 2021

    Salut @colin !

    Ce sont les membres du projet qui ont les droits d'écritures sur la branche de destination (dans ton cas la branche dev_cooperatic).

    Pour info, on dit plutôt « merger » la « merge requests » que « fermer » 🙂

    Salut @colin ! Ce sont [les membres du projet](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/project_members) qui ont les droits d'écritures sur la branche de destination (dans ton cas la branche `dev_cooperatic`). Pour info, on dit plutôt « merger » la « merge requests » que « fermer » :slight_smile:
  • Paul @paul_lgds commented May 18, 2021

    cc @fracolo

    cc @fracolo
  • Colin MORAND @colin commented May 18, 2021
    Developer

    Je utilisais "fermé" car sur le bouton de dessous il y a "Close merge request" ;-)

    Je utilisais "fermé" car sur le bouton de dessous il y a "Close merge request" ;-)
  • François C. @fracolo commented May 19, 2021
    Master

    Oups.... Je n'avais pas vu les notifications.

    D'accord avec @paul_lgds pour l'utilisation de fichiers de "traduction" pour différencier ce qui relève du "paramétrage" des messages affichés dans l'interface.

    Oups.... Je n'avais pas vu les notifications. D'accord avec @paul_lgds pour l'utilisation de fichiers de "traduction" pour différencier ce qui relève du "paramétrage" des messages affichés dans l'interface.
  • François C. @fracolo

    merged

    May 19, 2021

    merged

    merged
    Toggle commit list
  • François C. @fracolo

    mentioned in commit 83db9990

    May 19, 2021

    mentioned in commit 83db9990

    mentioned in commit 83db9990c6f96962f79054558befb9b17288d6c4
    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
François C.
Assignee
François C. @fracolo
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: cooperatic-foodcoops/third-party!14
×

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.