[18.0][MIG] base_external_system: Migration to 18.0#447
Conversation
6ca286d to
595a083
Compare
|
@NL66278 could you please do a rebase? Thank you! |
595a083 to
7abd05e
Compare
|
The original contents of the PR where a full migration of the improved 16.0 version, that unfortunately was never merged. Now that the unimproved 16.0 version has been merged into 18.0, I freshly reïmplemented the 16.0 improvements. |
Kiplangatdan
left a comment
There was a problem hiding this comment.
@NL66278 There are some obsolete files no longer used, otherwise this looks good:
../models/external_system_os.py
../tests/test_external_system_os.py
|
This PR has the |
@Kiplangatdan Those files are really not there. Maybe in your working directory, but not in the PR. Please have another look. |
@NL66278 It doesn't show in the PR because they were never touched: |
7abd05e to
42eaa6d
Compare
|
@Kiplangatdan Ah, I got confused because this was originally part of a migration branch. Files have now been removed. |
|
/ocabot migration base_external_system /ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 06081b2. Thanks a lot for contributing to OCA. ❤️ |
The module base_external_system will be the base for other modules to be migrated: