Skip to content

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 ;)

@quentinDupont quentinDupont force-pushed the 12.0_IMP_Invoice_fields_VAT branch from 41b28f2 to 39fbd99 Compare January 24, 2022 09:21
@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