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

[16.0][ADD] sale_import_base: add new module (clean history) #32

Draft
wants to merge 58 commits into
base: 16.0
Choose a base branch
from

Conversation

clementmbr
Copy link
Member

@clementmbr clementmbr commented Jan 22, 2025

Replace #21 with only sale_import_base commits and clean history.

sale_import_base depends on modules sale_channel_partner (#19) and queue_job_chunk (OCA/queue#737).

kevinkhao and others added 30 commits January 22, 2025 12:40
…idating cache

Test improvements on delivery carrier line generation
Co-authored-by: beau sebastien <sebastien.beau@akretion.com>
Co-authored-by: beau sebastien <sebastien.beau@akretion.com>
- add a specific validator for customer (as required field are not the
same)
- remove currency as it's deduce from pricelist
- remove useless validate_misc due to previous change
…odels: includes SO generation, tests, components, demo data

Add tests for cases where data content is full/minimal/mixed
Fix state_code: search also according to country_id
Fix state_code, country_code validation and search
…d raise on first error

[IMP] Validate payment currency as it should match Sale Order's pricelist's currency
kevinkhao and others added 26 commits January 22, 2025 12:40
…e numbering

Minor typo fix
Minor test refactor
…make sale.import.service.sale an Abstract Model
…erge and always have 2 distinct addresses

In case you want to update the delivery adresse after the creation of the sale order, it is easier to already have distinct addresses between delivery and invoicing.
Else one should create a new address, update the sale order, update the delivery order...
In the future, it  could become an option on channel or company instead of this hook that require an override to change the behavior
@clementmbr
Copy link
Member Author

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.

6 participants