Skip to content

Conversation

@jedel-odoo
Copy link

@jedel-odoo jedel-odoo commented Nov 4, 2025

The new legislation in France imposes to add te deliver date on the invoice.

task-5231287

@robodoo
Copy link

robodoo commented Nov 4, 2025

This PR targets the un-managed branch odoo-dev/odoo:18.0-rd-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

Commit message as usual and few change that could be done 😄

def _compute_show_delivery_date(self):
super()._compute_show_delivery_date()
for move in self:
if move.country_code == 'FR':

Choose a reason for hiding this comment

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

let's use l10n_fr_is_company_french instead, check why it's better 👀

The new legislation in France imposes to add te deliver date on the invoice.

task-5231287
@jedel-odoo jedel-odoo force-pushed the 18.0-rd-accounting-onboarding-delivery-date-jedel branch from 289da99 to a2112b3 Compare November 5, 2025 13:11
@jedel-odoo jedel-odoo changed the title [IMP] account: add the delivery date on invoices for France [IMP] account: add delivery date on invoices for France Nov 5, 2025
Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

Now what if the user confirm the move and that the delivery date is empty ? We could find a way to fill the information directly when posting the move 👀

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.

4 participants