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

[REM] theme_common, theme_*: remove theme_common #798

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

bso-odoo
Copy link
Collaborator

@bso-odoo bso-odoo commented Apr 9, 2024

Theme common used to contain tools that were reused by most of the themes. It has now become an empty module which pollutes the module dependency tree.

This commit removes theme_common and adapts themes that used to depend on it.

task-3696867

@robodoo
Copy link
Collaborator

robodoo commented Apr 9, 2024

theme_*: theme_bewise, theme_enark, theme_graphene, theme_treehouse

This commit copies the `theme_common` mixins in the few places where
they are used, to prepare for the future deletion of `theme_common`.

task-3696867
This commit removes the old snippets across all themes.

task-3696867
This commit removes old compatibility assets from all themes.

task-3696867
Since no mixin is used anymore, the file can be deleted.

task-3696867
This commit removes static files that are not referenced from an asset,
static files which are empty or only contain comments and
`primary_variables.scss` which only a never referenced variable.

task-3696867
This commit removes the whole mechanism about specifying the
`o-color-palette-number` through assets. It can still be done in the
custom SCSS.

task-3696867
Theme common used to contain tools that were reused by most of the
themes. It has now become an empty module which pollutes the module
dependency tree.

This commit removes `theme_common` and adapts themes that used to depend
on it.

task-3696867
@bso-odoo
Copy link
Collaborator Author

Related upgrade PR: https://github.com/odoo/upgrade/pull/5906

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.

2 participants