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

Price displayed in Odoo and PDF #161

Open
wants to merge 4 commits into
base: 12.0
Choose a base branch
from

Conversation

quentinDupont
Copy link
Member

@quentinDupont quentinDupont commented Apr 27, 2021

IMP sur les SO et Facture cliente :

  • Consistence entre la colonne affichée dans Odoo backoffice et les PDF
  • Meilleure lisibilité : la colonne Sous-total (backoffice et PDF) affiche si c'est HT ou TTC

Par défaut, si pas de position fiscale dans le SO et facture, on affiche la colonne Sous-Total (HT)

deck : apps/deck/#/board/144/card/1236

@quentinDupont quentinDupont changed the title [WIP] Odoo : colonnes marchent, PDF : marche sauf si pas de position … [IMP] Price displayed in Odoo and PDF Apr 28, 2021
@quentinDupont quentinDupont changed the title [IMP] Price displayed in Odoo and PDF [WIP] Price displayed in Odoo and PDF Apr 28, 2021
@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #161 (41b28f2) into 12.0 (52346b4) will increase coverage by 0.31%.
The diff coverage is 38.09%.

❗ Current head 41b28f2 differs from pull request most recent head 39fbd99. Consider uploading reports for the commit 39fbd99 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             12.0     #161      +/-   ##
==========================================
+ Coverage   72.79%   73.11%   +0.31%     
==========================================
  Files          94       87       -7     
  Lines         794      755      -39     
==========================================
- Hits          578      552      -26     
+ Misses        216      203      -13     
Impacted Files Coverage Δ
grap_qweb_report/models/sale_order.py 35.29% <27.27%> (-27.21%) ⬇️
grap_qweb_report/models/account_invoice.py 50.00% <50.00%> (-50.00%) ⬇️
grap_cooperative/models/res_company.py
...hange_views_account/models/account_account_type.py
grap_qweb_report/models/purchase_order.py
grap_qweb_report/models/report_custom_message.py
...rap_change_translation/models/res_country_group.py
..._qweb_report/models/report_custom_message_mixin.py
grap_qweb_report/models/stock_picking.py
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fede17c...39fbd99. Read the comment docs.

@quentinDupont quentinDupont changed the title [WIP] Price displayed in Odoo and PDF Price displayed in Odoo and PDF Apr 29, 2021
@legalsylvain
Copy link
Member

yop ! c'est chouette.

je suis dubitatif sur le cas pas de position fiscale, qui est finalement le cas qui va arriver le plus souvent.

Par défaut, si pas de position fiscale dans le SO et facture, on affiche la colonne Sous-Total (HT)

j'ai l'impression que ça va être reloud pour la majorité des actis en B2B.

qu'en penses-tu ?

bise.

@quentinDupont
Copy link
Member Author

yop ! c'est chouette.

je suis dubitatif sur le cas pas de position fiscale, qui est finalement le cas qui va arriver le plus souvent.

Par défaut, si pas de position fiscale dans le SO et facture, on affiche la colonne Sous-Total (HT)

j'ai l'impression que ça va être reloud pour la majorité des actis en B2B.

qu'en penses-tu ?

bise.

Actuellement pour les actis en B2B (et en B2C), si pas de position fiscale :
image
Le Sous-Total est d'ores et déjà en HT. Mais le PU en TTC, ce qui est très confusant vu qu'aucune des colonne n'est explicité.

Donc pour moi, ça clarifie juste ce qui est déja en place.
Aussi, genre chez ATE, tous ses client·es sont en bonne position fiscale, je pense qu'il y a un travail de bonne config à mener c'est certain, mais c'est en plus du scope de cet PR :)

@quentinDupont
Copy link
Member Author

@legalsylvain I remove try/except and learn from you ;)

@legalsylvain legalsylvain added this to the 12.0 milestone Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants