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.

Sign up for GitHub

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

Jump to bottom

[16.0][MIG] base_import_odoo #3136

Merged
merged 46 commits into from
Dec 12, 2024
Merged

[16.0][MIG] base_import_odoo #3136

merged 46 commits into from
Dec 12, 2024

Conversation

thomaspaulb
Copy link
Contributor

@thomaspaulb thomaspaulb commented Dec 4, 2024 •
edited
Loading

Continuation of #3004

hbrunn and others added 30 commits December 4, 2024 11:11
@hbrunn @thomaspaulb
[ADD] base_import_odoo
4fb2876
@hbrunn @thomaspaulb
[ADD] allow mapping records by equal field values
4849b6d
@hbrunn @thomaspaulb
[FIX] support firefox
816f3ae
@hbrunn @thomaspaulb
[IMP] community review
c22803f
@hbrunn @thomaspaulb
[RFR] use odoorpc
0cf62ca
@hbrunn @thomaspaulb
[ADD] make it possible to import res_model,res_id references
8a1e22d
@hbrunn @thomaspaulb
[IMP] disable recomputation during import
d304d40
@hbrunn @thomaspaulb
[ADD] better progress
9543726
@hbrunn @thomaspaulb
Revert "[IMP] disable recomputation during import"
244cb5d 
This reverts commit efce253.
@hbrunn @thomaspaulb
[ADD] more tests
781aaa7
@hbrunn @thomaspaulb
[FIX] dummy cleanup
75063ae 
[ADD] debug logging
@hbrunn @thomaspaulb
[ADD] depend on mail
19ce068
@hbrunn @thomaspaulb
[ADD] fix when running tests with accounting
565ed0e
@hbrunn @thomaspaulb
[ADD] duplicate treatment
3f5e019
@hbrunn @thomaspaulb
[FIX] don't pass an empty string for required char fields
fa4d93e
@StefanRijnhart @thomaspaulb
[FIX] Be less alarming about missing library (Module does not have to…
16d4d2e 
… be installed)
@yvaucher @thomaspaulb
Bump base_import_odoo version
0f2a696
@hbrunn @thomaspaulb
[ADD] create an xmlid for mapped records too (OCA#1059)
216b716 
* [ADD] create an xmlid for mapped records too
* [IMP] filter out unknown fields
* [FIX] warning about exception handling
@hbrunn @thomaspaulb
[ADD] [base_import_odoo] don't write falsy values on required fields
0b8e9f8 
with a default value
@hbrunn @thomaspaulb
[MIG] base_import_odoo
fed8758
@hbrunn @thomaspaulb
[IMP] base_import_odoo: black, isort, prettier
2a111a4
@hbrunn @thomaspaulb
[MIG] base_import_odoo: Migration to v13
70d4fec
@hbrunn @thomaspaulb
[ADD] possibility to keep passwords
8d11af2
@hbrunn @thomaspaulb
[FIX] don't scope xmlids with the module's name to avoid issues on un…
f0dc16c 
…install
@thomaspaulb
[IMP] base_import_odoo: black, isort, prettier
1ac870b
@thomaspaulb
[MIG] base_import_odoo: Migration to 14.0
dc6cb1c
@thomaspaulb
[UPD] for a reference id field, also allow many2one_reference, not on…
f866599 
…ly integer
@thomaspaulb
[IMP] dont try to query fields that dont exist remotely
088e5bb
@thomaspaulb
[IMP] default to 'id asc' to create records in the same order as they…
a3cc81c 
… were created in the other db
@thomaspaulb
[ADD] Possibility to query for some extra fields, that you can then p…
0d67e2b 
…ostprocess
@thomaspaulb thomaspaulb marked this pull request as ready for review December 4, 2024 11:35
@len-foss len-foss mentioned this pull request Dec 4, 2024
Closed
hbrunn
hbrunn approved these changes Dec 4, 2024
View reviewed changes
Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

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

I think it's good to have maintainers recorded on modules, would you mind adding yourself and me?

@thomaspaulb thomaspaulb force-pushed the 16.0-mig-base_import_odoo branch from f962f21 to 95727ee Compare December 5, 2024 10:13
@thomaspaulb
Copy link
Contributor Author

@hbrunn Added

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@hbrunn
Copy link
Member

hbrunn commented Dec 12, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3136-by-hbrunn-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 12, 2024
@OCA-git-bot
Merge PR #3136 into 16.0
0600bfa 
Signed-off-by hbrunn
@OCA-git-bot
Copy link
Contributor

@hbrunn your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-3136-by-hbrunn-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@thomaspaulb
Copy link
Contributor Author 

2024-12-12 12:32:49,277 415 INFO odoo odoo.addons.auditlog.tests.test_multi_company: ====================================================================== 
2024-12-12 12:32:49,277 415 ERROR odoo odoo.addons.auditlog.tests.test_multi_company: FAIL: TestMultiCompany.test_group_set_users_with_auditlog
Traceback (most recent call last):
  File "/__w/server-tools/server-tools/auditlog/tests/test_multi_company.py", line 113, in test_group_set_users_with_auditlog
    self.test_group_set_users()
  File "/__w/server-tools/server-tools/auditlog/tests/test_multi_company.py", line 86, in test_group_set_users
    self.assertEqual(group_with_user.users, self.user2)
AssertionError: res.users(11, 12) != res.users(12,)

:-/

@hbrunn
Copy link
Member

hbrunn commented Dec 12, 2024

fixed in #3146

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3136-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b95dd4a into OCA:16.0 Dec 12, 2024
9 checks passed
@OCA-git-bot
Copy link
Contributor

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

@len-foss len-foss len-foss approved these changes

@MiquelRForgeFlow MiquelRForgeFlow MiquelRForgeFlow approved these changes

@hbrunn hbrunn hbrunn approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@thomaspaulb @OCA-git-bot @hbrunn @pedrobaeza @MiquelRForgeFlow @len-foss @StefanRijnhart @yvaucher @ntsirintanis @mymage