Skip to content

[17.0][MIG] purchase_sale_inter_company#604

Merged
OCA-git-bot merged 70 commits intoOCA:17.0from
komit-consulting:17.0-mig-purchase_sale_inter_company
May 12, 2025
Merged

[17.0][MIG] purchase_sale_inter_company#604
OCA-git-bot merged 70 commits intoOCA:17.0from
komit-consulting:17.0-mig-purchase_sale_inter_company

Conversation

@cuongnmtm
Copy link
Contributor

@cuongnmtm cuongnmtm commented Mar 26, 2024

@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch from fd9e02d to 24f2291 Compare March 26, 2024 14:26
@cuongnmtm
Copy link
Contributor Author

cuongnmtm commented Mar 27, 2024

[FIXED] TODO: Fix this

image

@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch from 24f2291 to 02d2ae6 Compare March 27, 2024 13:50
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 28, 2024
@pomazanbohdan
Copy link

/ocabot migration purchase_sale_inter_company

@OCA-git-bot
Copy link
Contributor

Sorry @pomazanbohdan you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@pomazanbohdan
Copy link

/ocabot migration purchase_sale_inter_company

@cuongnmtm
Would you be so kind as to repeat the note to the bot?

@cuongnmtm
Copy link
Contributor Author

@pedrobaeza please help to trigger /ocabot migration purchase_sale_inter_company, thank you.

@OCA-git-bot
Copy link
Contributor

Sorry @cuongnmtm you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@cuongnmtm
Copy link
Contributor Author

cuongnmtm commented Aug 1, 2024

@pedrobaeza please help to trigger /ocabot migration purchase_sale_inter_company, thank you.

@OCA-git-bot
Copy link
Contributor

Sorry @cuongnmtm you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@cuongnmtm
Copy link
Contributor Author

TODO: Fix this
image

Probably I didn't work on this, I will check it soon.

@pedrobaeza
Copy link
Member

@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch 2 times, most recently from cdcd1f9 to 32cadba Compare August 2, 2024 11:20
@cuongnmtm
Copy link
Contributor Author

@pedrobaeza I fixed the pre-commit issues and added a reference to another pull request. For the code coverage, I tried adding two automated tests to improve it, but it somehow required writing tests for the tests...

image

@pedrobaeza
Copy link
Member

OK, I think it can be enough for now.

/ocabot migration purchase_sale_inter_company

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Aug 2, 2024
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 4, 2024
@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch 2 times, most recently from 0c123db to 0094dda Compare September 6, 2024 12:08
@JordiMForgeFlow
Copy link
Contributor

@cuongnmtm could you add #721 in the migration?

@cuongnmtm
Copy link
Contributor Author

JordiMForgeFlow

I cherry-picked your commit.

@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch 2 times, most recently from 0def0bc to a52f752 Compare December 6, 2024 18:30
@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch from a52f752 to 9e2d392 Compare December 20, 2024 09:39
Kev-Roche and others added 20 commits May 12, 2025 19:20
This is the splitted module from original to remove dependency on stock
Pricelist currency should be same with PO/SO document
If corresponding pricelist currency is not the same find or create propriate pricelist
Company contacts can be Vendors to create SO from PO
If purchase_exception or sale_exception modules are installed
we have an issue in tests, because order's confirm methods won't be called.
Possible fix is to set ignore_exception=True on orders in tests.
Exception check could be skipped by setting check_exception=False,
as mentioned in base_exception, but in fact it doesn't work.
Pricelist is set with the company
Update v14 latest changes related to test case failing

[UPD] Update purchase_sale_inter_company.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: multi-company-15.0/multi-company-15.0-purchase_sale_inter_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-15-0/multi-company-15-0-purchase_sale_inter_company/
Currently translated at 75.0% (15 of 20 strings)

Translation: multi-company-15.0/multi-company-15.0-purchase_sale_inter_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-15-0/multi-company-15-0-purchase_sale_inter_company/it/
Currently translated at 100.0% (20 of 20 strings)

Translation: multi-company-16.0/multi-company-16.0-purchase_sale_inter_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-16-0/multi-company-16-0-purchase_sale_inter_company/es/
…pdates

When the purchase user updates the PO after the SO is generated and
always if both are confirmed and unlocked, we sync the new lines to update the demand.

TT46773
Currently translated at 100.0% (25 of 25 strings)

Translation: multi-company-16.0/multi-company-16.0-purchase_sale_inter_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-16-0/multi-company-16-0-purchase_sale_inter_company/es/
…duct

to purchase order line for better extensibility
Currently translated at 100.0% (25 of 25 strings)

Translation: multi-company-16.0/multi-company-16.0-purchase_sale_inter_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-16-0/multi-company-16-0-purchase_sale_inter_company/it/
@cuongnmtm cuongnmtm force-pushed the 17.0-mig-purchase_sale_inter_company branch from 841bb03 to 5d28385 Compare May 12, 2025 12:21
@cuongnmtm
Copy link
Contributor Author

I have squashed more administrative commits. Thanks @carlos-lopez-tecnativa for your suggestion.

@carlos-lopez-tecnativa
Copy link
Contributor

@pedrobaeza could you please review this

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-604-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8fc06a6 into OCA:17.0 May 12, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a5aca7c. 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.