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

[15.0][FWD] connector_importer: misc fix/imp #126

Merged
merged 16 commits into from
Aug 23, 2023

Conversation

simahawk
Copy link
Contributor

Ports from 14.0

You can now configure a custom domain on record_handler options.

Eg:

options:
  record_handler:
    match_domain: "[('name', =, values['name'])]"

When defined, it will take precedence over the unique key if any.
Before this change if you did not override 'odoo_find' itself
your custom odoo_find_domain would be bypassed
if you had an empty unique_key.

This was a clear issue when you specified only
'options.record_handler.match_domain'.
Values where not properly converted for x2m fields.
You can now use ref and env to compute your domain.
Keys having both prefix and a transformer like 'xid::' were discarded.
A key like 'xid::foo.parent_id' would simply be ignored.
You can now pass options for specific fields converter via conf.
If for instance, the model you are importing has a relation `partner_id`
you can state that missing records must be created automatically.

Eg::

- model: product.product
  options:
    importer:
      odoo_unique_key: barcode
    mapper:
      name: product.product.mapper
      converter:
        categ_id:
          create_missing: True

All the keys inside converted/field will be propagated to the
`backend_to_rel` converter.
To ease retrieval of import session reports.
You can now pass any context key to the importer (propagated to the record handler)

by using 'ctx' key in the 'importer' options.
Was broken because available_importers requires an import type which is not available before editing.
It was also impossible to set the backend directly.
@simahawk
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-126-by-simahawk-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 58f96c2 into OCA:15.0 Aug 23, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

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

2 participants