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 2
    • Merge Requests 2
  • 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
  • !26

Merged
Opened Jun 09, 2021 by Arnaud B@ArnaudBoy 
  • Report abuse
Report abuse

Ticket 820 - suppression de la colonne Autres pour La Cagette uniquement lors de la réception

  • Discussion 1
  • Commits 2
  • Pipelines 2
  • Changes 4
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • François C.
    @fracolo started a discussion on an old version of the diff Jun 09, 2021
    Resolved by Arnaud B Jun 11, 2021
    reception/views.py
    76 76 def produits(request, id):
    77 77 """ Gets Order details """
    78 78 context = {'title': 'Réception des produits',
    79 "TOOLS_SERVER": settings.TOOLS_SERVER}
    79 "TOOLS_SERVER": settings.TOOLS_SERVER,
    80 "DISPLAY_AUTRES": "false" if (settings.COMPANY_NAME == "La Cagette") else "true",
    • François C. @fracolo commented Jun 09, 2021
      Master

      En l'état cela évite d'avoir à modifier les fichiers config.py sur les instances. Mais, il me semble préférable d'éviter de tester un nom de coop dans le code. Dans des situations équivalentes, j'ai utilisé une variable qui peut ou pas être déclarée dans le fichier config.py Dans le code la variable est initialiser par : lavariable = getattr(settings, 'NOM_VAR_DANS_CONFIG', Valeur_par_defaut) Dans le cas présent, toutes les coops, sauf la Cagette auront la valeur par défaut et seul le fichier config.py devra être adapté.

      Edited Jun 11, 2021
      En l'état cela évite d'avoir à modifier les fichiers config.py sur les instances. Mais, il me semble préférable d'éviter de tester un nom de coop dans le code. Dans des situations équivalentes, j'ai utilisé une variable qui peut ou pas être déclarée dans le fichier config.py Dans le code la variable est initialiser par : `lavariable = getattr(settings, 'NOM_VAR_DANS_CONFIG', Valeur_par_defaut)` Dans le cas présent, toutes les coops, sauf la Cagette auront la valeur par défaut et seul le fichier config.py devra être adapté.
    • Arnaud B @ArnaudBoy

      changed this line in version 2 of the diff

      Jun 11, 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/26/diffs?diff_id=738&start_sha=5959ace7268d55509345ff4419017830b72cfb5a#bdbf4bf517cfbcb11c21d01a57d214969543fced_80_80)
      Toggle commit list
    Please register or sign in to reply
  • Arnaud B @ArnaudBoy

    added 1 commit

    • 3383ef83 - Utilisation d'une variable pour afficher / masquer la colonne Autres

    Compare with previous version

    Jun 11, 2021

    added 1 commit

    • 3383ef83 - Utilisation d'une variable pour afficher / masquer la colonne Autres

    Compare with previous version

    added 1 commit <ul><li>3383ef83 - Utilisation d&#39;une variable pour afficher / masquer la colonne Autres</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/26/diffs?diff_id=738&start_sha=5959ace7268d55509345ff4419017830b72cfb5a)
    Toggle commit list
  • Arnaud B @ArnaudBoy

    resolved all discussions

    Jun 11, 2021

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • François C. @fracolo

    merged

    Jun 17, 2021

    merged

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

    mentioned in commit 2727fc54

    Jun 17, 2021

    mentioned in commit 2727fc54

    mentioned in commit 2727fc54862cae42692eff25d26c4fa211ad575b
    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: cooperatic-foodcoops/third-party!26
×

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.