[13.0][MIG] account_invoice_inter_company #196
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* [10.0]Migrate account_invoice_inter_company module to v10 * refactor code for play onchange method * active demo datas and fix account_invoice.py, res_config.py, README.rst file * extract the search domain of users from the _check_intercompany_product method * fix test * fix test and update french translation * rename openerp to odoo * fix test_flake8 and fix views after David's comment * add oca_dependencies.txt * fix bare except * improve action_cancel method * fix chart_template_id in tests * fix transfer_account_id in tests * fix chart_template_id for company B in tests * improve tests * fix tests taking into account the base_multi_company module * fix tests and contributors after lasley's comment * improve tests * fix account_invoice.py after mourad's comment * fix tests after lasley's comment * fix transfer account template in demo data * increases test coverage * improve action_invoice_open method after astirpe's comment * fix value for chart_template_id in tests
This demo data is quite dangerous by design, since it creates compenies, accounts, decides the companies country, etc. All of that by hand. In a tested integration environment, this fails when installing account-related addons due to companies not having a chart of accounts. The fix is simple: remove this from demo data and leave it just as test data.
… in invoice lines (OCA#111) * Add product mapping in roadmap * Allow creation of invoice lines without products
FIX misc annotations from @astirpe, thanks Co-Authored-By: bealdav <david.beal@akretion.com>
Co-Authored-By: mourad-ehm <mourad.elhadj.mimoune@akretion.com>
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: multi-company-12.0/multi-company-12.0-account_invoice_inter_company Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-account_invoice_inter_company/
…rity `password_security` asks by default for some minimal rules in the password. If it happens to be installed before this module, in an integration environment, it cannot be loaded. It's just demo data, so I just use a longer password here.
Currently translated at 100.0% (22 of 22 strings) Translation: multi-company-12.0/multi-company-12.0-account_invoice_inter_company Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-account_invoice_inter_company/zh_CN/
Currently translated at 50.0% (11 of 22 strings) Translation: multi-company-12.0/multi-company-12.0-account_invoice_inter_company Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-account_invoice_inter_company/hr/
Currently translated at 100.0% (22 of 22 strings) Translation: multi-company-12.0/multi-company-12.0-account_invoice_inter_company Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-account_invoice_inter_company/it/
AdriaGForgeFlow
force-pushed
the
13.0-mig-account_invoice_inter_company
branch
from
6ce05ca
to
6a37538
Compare
LoisRForgeFlow
approved these changes
Mar 30, 2020
AaronHForgeFlow
approved these changes
Mar 30, 2020
| <field name="price_unit" eval="450.0" /> | ||
| <field name="company_id" ref="company_a" /> | ||
| </record> | ||
| <!-- <record id="customer_invoice_company_a" model="account.move">--> |
There was a problem hiding this comment.
Better to remove this?
There was a problem hiding this comment.
Yes, you're right I missed this one.
AdriaGForgeFlow
force-pushed
the
13.0-mig-account_invoice_inter_company
branch
from
6a37538
to
ac860b2
Compare
1 task
JordiBForgeFlow
force-pushed
the
13.0-mig-account_invoice_inter_company
branch
2 times, most recently
from
db910d8
to
0b81066
Compare
* intercompany_invoice_user_id * company_share_product (restored from previous versions)
JordiBForgeFlow
force-pushed
the
13.0-mig-account_invoice_inter_company
branch
from
0b81066
to
b173f0b
Compare
|
@AdriaGForgeFlow after reviewing the purchase_sale_inter_company and the previous versions of this module it seems to me that we should better define an inter_company_invoice_user_id, and also we need to add to the config the option to disable the product security rules. Implemented here: |
JordiBForgeFlow
approved these changes
Mar 31, 2020
Makes sense, thanks for the help 👍 |
AdriaGForgeFlow
force-pushed
the
13.0-mig-account_invoice_inter_company
branch
from
af56c17
to
ace654c
Compare
|
/ocabot merge |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 334c378. Thanks a lot for contributing to OCA. ❤️ |
SilvioC2C
added a commit
to camptocamp/multi-company
that referenced
this pull request
Mar 7, 2023
* Fix releases 15.0.1.0.z (OCA#183) * BSSBG-2749: Remove cloud-platform modules from migration scripts * BSSBG-2749: Add end-of-migration script to introduce pre-mig releases * BSSBG-2749: Squash all migration-related releases * Release 15.0.1.0.3 * BSSBG-2781: Upgrade Odoo core and Enterprise submodules * BSSBG-2781: Remove deprecated platform script * Release 15.0.1.0.4 * Fix: Remove ``website`` and related modules from migration (OCA#196) * Fix: Fix menu icons and remove Odoo upgrade banner and ribbon (OCA#199) * Fix: Fix wrong migration script for product's ``quality_check`` field * Fix: Fix wrongly declared view inheritance on ``product.template`` * Imp: Update on-premise ``docker-compose.yml`` file (OCA#201) * Release 15.0.1.0.5
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Standard migration of module account_invoice_inter_company to v13.
cc ~ @ForgeFlow