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

[18.0] [MIG] sale_order_customer_free_ref #1126

Merged
merged 16 commits into from
Feb 14, 2025

Conversation

rlimaeco
Copy link

No description provided.

TDu and others added 15 commits February 11, 2025 14:00
The goal of this module is to improve on the `client_order_ref` on `sale.order`.

By default Odoo only has one field to handle the customer reference of a sale order.
But when using EDI some specifications allow to have two fields for this, the customer
order Id and a customer (free) reference.

To help with this, this module adds two specific fields for them and transform the
`client_order_ref` standard field into a computed one.

The two new fields are also passed on to created invoices.
Currently translated at 100.0% (8 of 8 strings)

Translation: edi-14.0/edi-14.0-sale_order_customer_free_ref
Translate-URL: https://translation.odoo-community.org/projects/edi-14-0/edi-14-0-sale_order_customer_free_ref/es/
@rlimaeco rlimaeco force-pushed the 18.0-mig-sale_order_customer_free_ref branch from 6f3d147 to f1eb734 Compare February 12, 2025 14:45
@rlimaeco
Copy link
Author

@simahawk and @mmequignon please review this PR

Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

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

small remark

@rlimaeco rlimaeco force-pushed the 18.0-mig-sale_order_customer_free_ref branch from f1eb734 to b98b5e1 Compare February 13, 2025 10:28
@rlimaeco rlimaeco requested a review from simahawk February 13, 2025 16:44
Copy link

@twalter-c2c twalter-c2c left a comment

Choose a reason for hiding this comment

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

LGTM

@simahawk
Copy link
Contributor

/ocabot migration sale_order_customer_free_ref
/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1126-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Feb 14, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 14, 2025
11 tasks
@OCA-git-bot OCA-git-bot merged commit ea53592 into OCA:18.0 Feb 14, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.

10 participants