Conversation
|
/ocabot migration base_import_async |
hoangtrann
left a comment
There was a problem hiding this comment.
Code LGTM, though you can now remove queue_job from test-requirements.txt
f42ea1a to
e83c07f
Compare
- Move BaseImportImport wizard code to wizard/\n- Update tests and async import flow\n- Update pre-commit exclusions
4416fe3 to
eba1ed2
Compare
|
Hi @hoangtrann @sbidoul , I rebased the branch on 19.0, resolved the conflicts, and addressed the review nits (removed the obsolete test-requirements.txt dependency and the “Odoo 19” comment). I also restored requirements.txt (generated from external_dependencies) and added tests to improve coverage. Finally, I squashed the PR into a single commit; CI is running again on the new head. Please double check if this is ready to get merged. |
|
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. |
Scope:
Depends on:
Summary:
Temporary references:
Pre-commit:
Tests: