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

Merged
Opened May 05, 2022 by François C.@fracolo 
  • Report abuse
Report abuse

Add change product shelf function

  • Discussion 10
  • Commits 3
  • Pipelines 3
  • Changes 8
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on the diff May 06, 2022
    Resolved by François C. May 06, 2022
    shelfs/models.py
    529 532 except Exception as e:
    530 533 coop_logger.error("Rayons, get_shelfs_sortorder : %s", str(e))
    531 534 return res
    535
    536 @staticmethod
    537 def make_products_shelf_links(data):
    • Damien Moulard @DamienM commented May 06, 2022
      Developer

      Je mettrais un commentaire pour préciser ce que fait cette fonction

      Je mettrais un commentaire pour préciser ce que fait cette fonction
    • François C. @fracolo commented May 06, 2022
      Master

      Fait

      Fait
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on the diff May 06, 2022
    Resolved by Damien Moulard May 06, 2022
    shelfs/static/js/shelf_inventory.js
    766 936
    767 937 /* INIT */
    768 938
    939 // (for shelf change)
    940 function get_all_shelfs() {
    • Damien Moulard @DamienM commented May 06, 2022
      Developer

      L'idéal aurait été de sauvegarder les rayons en local vu qu'on les récupère tous à l'écran d'avant, ça fait plusieurs fois que j'y pense. Mais en l'état , ça fonctionne bien.

      L'idéal aurait été de sauvegarder les rayons en local vu qu'on les récupère tous à l'écran d'avant, ça fait plusieurs fois que j'y pense. Mais en l'état , ça fonctionne bien.
    • François C. @fracolo commented May 06, 2022
      Master

      Les rayons peuvent changer : les stocker en local voudrait dire de faire une synchro dès que nécessaire (sur quel critère ?)

      Les rayons peuvent changer : les stocker en local voudrait dire de faire une synchro dès que nécessaire (sur quel critère ?)
    • Damien Moulard @DamienM commented May 06, 2022
      Developer

      Pas faux !

      Pas faux !
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff May 06, 2022
    Resolved by Damien Moulard May 06, 2022
    shelfs/models.py
    529 532 except Exception as e:
    530 533 coop_logger.error("Rayons, get_shelfs_sortorder : %s", str(e))
    531 534 return res
    535
    536 @staticmethod
    537 def make_products_shelf_links(data):
    538 res = {}
    539 try:
    540 api = OdooAPI()
    541 res['done'] = []
    542 for elt in data:
    543 f = {'shelf_id': elt['shelf_id']}
    544 if api.update('product.product', [elt['product_id']], f):
    • Damien Moulard @DamienM commented May 06, 2022
      Developer

      À coup d'une opération par produit, ça risque d'être long s'il y a beaucoup de produits à mettre à jour non ? Pour parer à ça on pourrait mettre à jour par batch tous les produits qui vont dans le même rayon.

      À coup d'une opération par produit, ça risque d'être long s'il y a beaucoup de produits à mettre à jour non ? Pour parer à ça on pourrait mettre à jour par batch tous les produits qui vont dans le même rayon.
    • François C. @fracolo commented May 06, 2022
      Master

      C'est juste ! Je fais les adaptations

      C'est juste ! Je fais les adaptations
    • François C. @fracolo

      changed this line in version 3 of the diff

      May 06, 2022

      changed this line in version 3 of the diff

      changed this line in [version 3 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/175/diffs?diff_id=1632&start_sha=e501da639cf7a3ea0547345b870cafa17b162d5b#2af5c595d30f82741787a232207c56808c90ce1d_545_547)
      Toggle commit list
    Please register or sign in to reply
  • François C. @fracolo

    added 1 commit

    • e501da63 - Add comment to make_products_shelf_links

    Compare with previous version

    May 06, 2022

    added 1 commit

    • e501da63 - Add comment to make_products_shelf_links

    Compare with previous version

    added 1 commit <ul><li>e501da63 - Add comment to make_products_shelf_links</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/175/diffs?diff_id=1629&start_sha=8a14e906f3b66687f6d28d1f79948abfd71fc58b)
    Toggle commit list
  • Damien Moulard @DamienM commented May 06, 2022
    Developer
    • Ce n'est pas précisé dans le ticket, mais j'aurais mis dans la dropdown les rayons triés par ordre alphabétique (en ayant quand même "Articles sans rayon" en premier), ça me semble plus simple à trouver comme ça vu que les coop ne connaissent pas les numéros de rayons. Qu'en penses-tu, est-ce que ça vaut le coup sachant que le but principal reste de mettre des produits dans "Articles sans rayon" ?
    • Je rajouterais une notification que l'opération a bien été réussie.
    * Ce n'est pas précisé dans le ticket, mais j'aurais mis dans la dropdown les rayons triés par ordre alphabétique (en ayant quand même "Articles sans rayon" en premier), ça me semble plus simple à trouver comme ça vu que les coop ne connaissent pas les numéros de rayons. Qu'en penses-tu, est-ce que ça vaut le coup sachant que le but principal reste de mettre des produits dans "Articles sans rayon" ? * Je rajouterais une notification que l'opération a bien été réussie.
  • François C. @fracolo commented May 06, 2022
    Master
    • Pour l'ordre : Les 2 premiers seront les plus couramment utilisés (pour la Cagette : sans rayon et en promo). Il est nécessaire d'ajouter un paramètre pour les identifiants de rayons à mettre en premiers, puis de trier par ordre alpha les suivants. Je fais les adaptations.
    • J'ajoute pendant les adaptations la notifications que l'opération a bien réussie.
    * Pour l'ordre : Les 2 premiers seront les plus couramment utilisés (pour la Cagette : sans rayon et en promo). Il est nécessaire d'ajouter un paramètre pour les identifiants de rayons à mettre en premiers, puis de trier par ordre alpha les suivants. Je fais les adaptations. * J'ajoute pendant les adaptations la notifications que l'opération a bien réussie.
  • François C. @fracolo

    added 1 commit

    • e42f5e6f - Modifications (see PR 175 comments)

    Compare with previous version

    May 06, 2022

    added 1 commit

    • e42f5e6f - Modifications (see PR 175 comments)

    Compare with previous version

    added 1 commit <ul><li>e42f5e6f - Modifications (see PR 175 comments)</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/175/diffs?diff_id=1632&start_sha=e501da639cf7a3ea0547345b870cafa17b162d5b)
    Toggle commit list
  • François C. @fracolo commented May 06, 2022
    Master

    Pour la Cagette, ajouter SHELFS_TO_BE_AHEAD_IN_SELECT_LIST = [90,74] dans config.py

    Pour la Cagette, ajouter SHELFS_TO_BE_AHEAD_IN_SELECT_LIST = [90,74] dans config.py
  • Damien Moulard @DamienM

    resolved all discussions

    May 06, 2022

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    May 06, 2022

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 26bae4b3

    May 06, 2022

    mentioned in commit 26bae4b3

    mentioned in commit 26bae4b3c64a75213efe5437e4644ad1be5d3f0b
    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!175
×

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.