Conversation
flachica
force-pushed
the
15.0-mig-base_multi_image
branch
2 times, most recently
from
7c2e4c5 to
d4d172b
Compare
|
@flachica Thanks for this. Could you update your last commit message in order to have something like '[15.0][MIG] base_multi_image'? |
…xisting attachment record + Fix the 'pre_init_hook_for_submodules()' hook to extract the images from the ir_attachment table for binary fields initialized with the 'attachment=True' parameter
…dule is uninstalled.
* [FIX] base_multi_image: Use env in uninstall_hook
improve uninstall hook to move images from multi to single mode
Currently translated at 100.0% (48 of 48 strings) Translation: server-tools-14.0/server-tools-14.0-base_multi_image Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-base_multi_image/es_AR/
flachica
force-pushed
the
15.0-mig-base_multi_image
branch
from
d4d172b to
265b006
Compare
Changed, thanks for your reviews |
|
/ocabot merge nobump |
|
Sorry @jacekmichalski you are not allowed to merge. To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons. If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the |
|
@pedrobaeza isn't it ready to be merged? Btw, what about OCA/product-attribute#1070 ? |
|
I miss a second review, but I suppose yours counts as second one. Please use the proper GitHub review mechanism. /ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 01440ce. Thanks a lot for contributing to OCA. ❤️ |
16 participants
Dependency below, please review
OCA/connector-prestashop#151
OCA/sale-workflow#2005
OCA/connector-ecommerce#70
#2334
OCA/product-attribute#1070
OCA/product-attribute#1069