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

[14.0][IMP] l10n_it_delivery_note: refactor delivery note creation by extracting value preparation into a separate method to allow inheriting #4615

Conversation

GSLabIt
Copy link
Contributor

@GSLabIt GSLabIt commented Feb 12, 2025

riorganizzare la creazione del ddt estraendo la preparazione dei valori in un metodo separato per consentire di ereditare la funzione e aggiungere eventuali altri campi i.e. le note che potrebbero arrivare dal picking out

@OCA-git-bot
Copy link
Contributor

Hi @MarcoCalcagni, @renda-dev, @aleuffre,
some modules you are maintaining are being modified, check this out!

@GSLabIt GSLabIt force-pushed the 14.0-imp-l10n_it_delivery_note-prefer_prepare_delivery_note branch from ddb1ac5 to c8ff07b Compare February 12, 2025 10:42
@GSLabIt GSLabIt changed the title [IMP] l10n_it_delivery_note: refactor delivery note creation by extracting value preparation into a separate method [14.0][IMP] l10n_it_delivery_note: refactor delivery note creation by extracting value preparation into a separate method Feb 12, 2025
@GSLabIt GSLabIt changed the title [14.0][IMP] l10n_it_delivery_note: refactor delivery note creation by extracting value preparation into a separate method [14.0][IMP] l10n_it_delivery_note: refactor delivery note creation by extracting value preparation into a separate method to allow inheriting Feb 12, 2025
Copy link
Contributor

@aleuffre aleuffre left a comment

Choose a reason for hiding this comment

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

Code review, LGTM

@aleuffre
Copy link
Contributor

L'unica cosa, il messaggio di commit deve essere max 80 caratteri

@GSLabIt GSLabIt force-pushed the 14.0-imp-l10n_it_delivery_note-prefer_prepare_delivery_note branch from c8ff07b to ff161c1 Compare February 13, 2025 09:08
Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

LGTM

@aleuffre
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-4615-by-aleuffre-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e539ac8. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot merged commit 51974d0 into OCA:14.0 Feb 13, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants