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] Mixed MR's #696

Closed
wants to merge 8 commits into from
Closed

[14.0] Mixed MR's #696

wants to merge 8 commits into from

Conversation

thomaspaulb
Copy link
Contributor

@thomaspaulb thomaspaulb commented Sep 17, 2024

No description provided.

thomaspaulb and others added 8 commits September 17, 2024 13:41
…tional.

In some cases people still want to modify the invoice.
- serial number product combined with regular product
- backorder created on serial number validation
Kit products (bom type = phantom) expand into several other products upon
picking creation. Before this commit, this caused the module to find
multiple destination moves per each move, since the matching was done
on sale order - purchases order link only. Here we add matching on product
also, which should fix this.
…o rebel module

For product_supplierinfo_inter_company, this introduces a hard requirement to configure
a pricelist on each supplier. This makes the tests of other modules fail because they do
not set these.

For stock_intercompany I don't know why it was set to rebel, it was done before
but not updated in the copier template, so just updating copier to reflect reality.
When a return is actioned on the SO side, also process the return on PO side.
Currently when the sale order line has received a discount, and the SO is confirmed,
the undiscounted price is copied over to the PO, leading to a price discrepancy
between the SO and PO. With this commit, discount is respected.
By default picking state is forced to Done when SO picking is done, but if picking
sync fails, eg if there is a mismatch in SO and PO move lines, user can't correct
anything since the picking is already set to Done.

This may be acceptable in cases where 100% of the syncs go OK but we make the
setting configurable to support situations where manual corrections are needed.
@OCA-git-bot
Copy link
Contributor

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

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.

2 participants