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

[IMP] web.assets_common warning removed #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xaviedoanhduy
Copy link

web.assets_common removed in https://github.com/odoo/odoo/pull/132190

run command: python -m odoo_module_migrate -d /home/trobz/code/oca/web/17.0 -m web_assets_warmup -i 14.0 -t 17.0 -fp

15:54:38   INFO        Creating new branch '17.0-mig-web_assets_warmup' ...
Switched to a new branch '17.0-mig-web_assets_warmup'
15:54:38   INFO        Getting latest changes from old branch
remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0 (from 0)
From github.com:OCA/web
 * branch                14.0       -> FETCH_HEAD
15:54:42   INFO        Run pre-commit
************* Module web_assets_warmup.__manifest__
web_assets_warmup/__manifest__.py:6: [C8106(manifest-version-format), ] Wrong Version Format "14.0.1.0.0" in manifest file. Regex to match: "(17\.0)\.\d+\.\d+\.\d+$"
15:55:07   INFO        Stage and commit changes done by pre-commit
[17.0-mig-web_assets_warmup ecb206d7b] [IMP] web_assets_warmup: pre-commit execution
 7 files changed, 49 insertions(+), 40 deletions(-)
15:55:07   INFO        [web_assets_warmup] Running migration from 14.0 to 17.0
15:55:07   INFO        Bump version to 17.0.1.0.0
15:55:07   WARNING     [17] Reference to 'web.assets_common'. This bundel has been removed from the commit, see https://github.com/odoo/odoo/pull/132190. File /home/trobz/code/oca/web/17.0/web_assets_warmup/models/ir_actions_report.py
15:55:07   INFO        Commit changes for web_assets_warmup. commit name '[MIG] web_assets_warmup: Migration to 17.0'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant