Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD] New module mrp_sale_grouped #330

Conversation

quentinDupont
Copy link
Member

@quentinDupont quentinDupont commented Jul 7, 2023

Répond aux besoins suivants :

  • Regrouper différentes ventes et pouvoir rapidement voir les articles commandés (seulement sur sale brouillon et envoyés, pas les confirmées)
  • Lien vers les ordre de prod liés aux ventes (ça sera pas visible chez Grap encore)
  • Avoir une vision du CA théorique
  • PDF de récap de ventes
  • PDF de récap de quoi produire, acheter
  • PDF de quantités par produits

4 fonctionnalités principales des 4 boutons rouges ici
image

Numéro 1
image

Numéro 3
image

Numéro 4
image
donne
image

image

ROADMAP :

  • besoin quasi similaire mais différent, la même chose mais pour la prod prévue (donc pas basé sur sale.order mais sur mrp.production) ?
  • Lien avec la prod, que ça puisse acter de lancer des prod / des achats ? → Raccourci fait pour ordre de prod → À creuser avec un paramétrage d'article bien fait (pas encore le cas à Grap)
  • Gérer mieux si on vent des articles d'A/R (et que du coup ça force à créer un BoM)

À faire :

  • Doc après retour
  • Tests python

@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Attention: 70 lines in your changes are missing coverage. Please review.

Comparison is base (24a11dd) 79.10% compared to head (b04e195) 78.45%.

❗ Current head b04e195 differs from pull request most recent head adaa1c5. Consider uploading reports for the commit adaa1c5 to get more accurate results

Files Patch % Lines
mrp_sale_grouped/report/report_sale_grouped.py 19.04% 34 Missing ⚠️
...e_grouped/wizard/bom_print_purchase_list_wizard.py 81.69% 7 Missing and 6 partials ⚠️
mrp_sale_grouped/models/mrp_sale_grouped.py 77.77% 9 Missing and 1 partial ⚠️
...rp_sale_grouped/wizard/x2m_matrix_grouped_sales.py 54.54% 4 Missing and 1 partial ⚠️
..._purchase/wizard/bom_print_purchase_list_wizard.py 90.00% 1 Missing and 1 partial ⚠️
mrp_sale_grouped/models/sale_order_line.py 71.42% 2 Missing ⚠️
mrp_sale_grouped/wizard/sale_grouped_wizard.py 90.90% 1 Missing and 1 partial ⚠️
...rp_sale_grouped/wizard/sale_grouped_wizard_line.py 80.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             12.0     #330      +/-   ##
==========================================
- Coverage   79.10%   78.45%   -0.65%     
==========================================
  Files         207      216       +9     
  Lines        2130     2284     +154     
  Branches      369      400      +31     
==========================================
+ Hits         1685     1792     +107     
- Misses        426      465      +39     
- Partials       19       27       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@quentinDupont quentinDupont force-pushed the 12.0-941-ADD-mrp_grouped_sale_production-NEW_Matrix_2d_module_for_sale branch from 6c6dc3a to ab172c6 Compare October 16, 2023 14:24
@quentinDupont quentinDupont marked this pull request as ready for review October 16, 2023 14:26
@quentinDupont quentinDupont changed the title [ADD][WIP] New module mrp_grouped_sale_production [ADD] New module mrp_sale_grouped Oct 18, 2023
Copy link
Member

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revue rapide.

  1. un petite remarque sur un nom de champ.
  2. est-ce que tu pourrais rajouter des tests pour le nouveau module mrp_sale_grouped ?

image

Ref : https://app.codecov.io/gh/grap/grap-odoo-custom/pull/330

Note : j'avais pas vu ta todo list avec écrit "faire test".

Sinon, ça a l'air cool, mais j'ai pas pris le temps de me plonger fonctionnellement dessus !

mrp_sale_grouped/models/sale_order.py Outdated Show resolved Hide resolved
@legalsylvain legalsylvain mentioned this pull request Nov 3, 2023
@legalsylvain
Copy link
Member

Est ce que tu peux rebase ? Les tests sont rouges.

@quentinDupont quentinDupont force-pushed the 12.0-941-ADD-mrp_grouped_sale_production-NEW_Matrix_2d_module_for_sale branch from 3e563f8 to adaa1c5 Compare December 7, 2023 08:38
@github-grap-bot github-grap-bot merged commit adaa1c5 into grap:12.0 Dec 13, 2023
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants