[15.0][MIG] web_widget_mermaid by dreispt · Pull Request #2763 · OCA/web

dreispt · 2024-03-09T15:18:14Z

Supersedes #2308

Using the last official version of module web_widget_mermaid as a base for the migration to v15.0

Using new asset definition via manifest and changed from static mermaid.js file to external CDN. Removed web_widget_mermaid_default_theme.scss and web_widget_mermaid_view.xml from mainfest (detailed description in upcoming commits)

This file is no longer needed since assets are now defined in manifest file.

Fixed ESLint messages. Due to issue with mermaid call, the definition has been moved to init and was deactivated for rule 'no-undef'. Inline comments has been moved to additional lines.

Since it seems like null values are no longer working for theme parameter, theme has been set to 'base' and theme config can completely move to web_widget_mermaid.js. web_widget_mermaid_default_theme.scss will be removed in upcoming commit.

SCSS file of default theme can be removed since the theme config moved to web_widget_mermaid.js.

• Add contributor: RoboHeart • Remove 'Upgrading' section from USAGE.rst [ADD] Missing setup files [UPD] Documentation files by oca-gen-addon-readme

dreispt · 2024-03-09T15:21:12Z

/ocabot merge nobump

OCA-git-bot · 2024-03-09T15:21:18Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-2763-by-dreispt-bump-nobump, awaiting test results.

Signed-off-by dreispt

OCA-git-bot · 2024-03-09T15:27:11Z

@dreispt The merge process could not be finalized, because command /tmp/tmp43e738b6/bin/python setup.py bdist_wheel --dist-dir /tmp/tmppjq7bw95 --bdist-dir /tmp/tmp5a2mi6an --python-tag py3 failed with output:

/tmp/tmp43e738b6/lib/python3.8/site-packages/setuptools_scm/_integration/setuptools.py:30: RuntimeWarning: 
ERROR: setuptools==57.5.0 is used in combination with setuptools_scm>=8.x

Your build configuration is incomplete and previously worked by accident!
setuptools_scm requires setuptools>=61

Suggested workaround if applicable:
 - migrating from the deprecated setup_requires mechanism to pep517/518
   and using a pyproject.toml to declare build dependencies
   which are reliably pre-installed before running the build tools

  warnings.warn(
error in setup command: /tmp/tmpnbue9nxg/setup/web_widget_mermaid/odoo/addons must contain exactly one installable Odoo addon dir, found []

dreispt · 2024-03-09T15:44:59Z

/ocabot merge nobump

OCA-git-bot · 2024-03-09T15:46:17Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-2763-by-dreispt-bump-nobump, awaiting test results.

Signed-off-by dreispt

OCA-git-bot · 2024-03-09T15:51:59Z

@dreispt The merge process could not be finalized, because command /tmp/tmp43e738b6/bin/python setup.py bdist_wheel --dist-dir /tmp/tmpjzp8d1px --bdist-dir /tmp/tmpdmfjrn_p --python-tag py3 failed with output:

/tmp/tmp43e738b6/lib/python3.8/site-packages/setuptools_scm/_integration/setuptools.py:30: RuntimeWarning: 
ERROR: setuptools==57.5.0 is used in combination with setuptools_scm>=8.x

Your build configuration is incomplete and previously worked by accident!
setuptools_scm requires setuptools>=61

Suggested workaround if applicable:
 - migrating from the deprecated setup_requires mechanism to pep517/518
   and using a pyproject.toml to declare build dependencies
   which are reliably pre-installed before running the build tools

  warnings.warn(
error in setup command: /tmp/tmp3afsumew/setup/web_widget_mermaid/odoo/addons must contain exactly one installable Odoo addon dir, found []

dreispt · 2024-05-16T08:08:59Z

/ocabot merge nobump

OCA-git-bot · 2024-05-16T08:59:07Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-2763-by-dreispt-bump-nobump, awaiting test results.

Signed-off-by dreispt

OCA-git-bot · 2024-05-16T09:05:36Z

@dreispt The merge process could not be finalized, because command /tmp/tmpnjquwd9s/bin/python setup.py bdist_wheel --dist-dir /tmp/tmpkm2hwieq --bdist-dir /tmp/tmp3hzqrm89 --python-tag py3 failed with output:

/tmp/tmpnjquwd9s/lib/python3.8/site-packages/setuptools_scm/_integration/setuptools.py:30: RuntimeWarning: 
ERROR: setuptools==57.5.0 is used in combination with setuptools_scm>=8.x

Your build configuration is incomplete and previously worked by accident!
setuptools_scm requires setuptools>=61

Suggested workaround if applicable:
 - migrating from the deprecated setup_requires mechanism to pep517/518
   and using a pyproject.toml to declare build dependencies
   which are reliably pre-installed before running the build tools

  warnings.warn(
error in setup command: /tmp/tmpee_c8pj3/setup/web_widget_mermaid/odoo/addons must contain exactly one installable Odoo addon dir, found []

idamonli · 2024-05-26T10:03:11Z

@dreispt Do you have plans to migrate this module to version 16.0?

bosd · 2024-05-27T13:13:48Z

Would like to test this, but runboat is not available :(

hbrunn · 2025-07-18T09:04:58Z

/ocabot rebase

OCA-git-bot · 2025-07-18T09:05:05Z

@hbrunn The rebase process failed, because command git rebase origin/15.0 failed with output:

Rebasing (1/9)
Rebasing (2/9)
Rebasing (3/9)
Rebasing (4/9)
Rebasing (5/9)
Rebasing (6/9)
Rebasing (7/9)
Rebasing (8/9)
Rebasing (9/9)
Auto-merging .copier-answers.yml
CONFLICT (content): Merge conflict in .copier-answers.yml
Auto-merging .gitignore
Auto-merging .pre-commit-config.yaml
CONFLICT (content): Merge conflict in .pre-commit-config.yaml
error: could not apply ddea6a9db... [REF] pre-commit: copier update
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply ddea6a9db... [REF] pre-commit: copier update

bosd

This does not seem to be working:

TheRealRoboHeart added 8 commits March 9, 2024 15:15

[ADD] web_widget_mermaid from v12.0

d91e202

Using the last official version of module web_widget_mermaid as a base for the migration to v15.0

[IMP] Add assets

c2d59cc

Using new asset definition via manifest and changed from static mermaid.js file to external CDN. Removed web_widget_mermaid_default_theme.scss and web_widget_mermaid_view.xml from mainfest (detailed description in upcoming commits)

[REM] Remove web_widget_mermaid_view.xml

7a5d838

This file is no longer needed since assets are now defined in manifest file.

[FIX] ESLint messages

b54c5fe

Fixed ESLint messages. Due to issue with mermaid call, the definition has been moved to init and was deactivated for rule 'no-undef'. Inline comments has been moved to additional lines.

[IMP] Add themeVariables to defaultConfig

3e084c8

Since it seems like null values are no longer working for theme parameter, theme has been set to 'base' and theme config can completely move to web_widget_mermaid.js. web_widget_mermaid_default_theme.scss will be removed in upcoming commit.

[REM] Remove web_widget_mermaid_default_theme.scss

6c0f6dd

SCSS file of default theme can be removed since the theme config moved to web_widget_mermaid.js.

[FIX] pre-commit issues (prettier)

2cc0980

[UPD] Readme fragments

c2e2378

• Add contributor: RoboHeart • Remove 'Upgrading' section from USAGE.rst [ADD] Missing setup files [UPD] Documentation files by oca-gen-addon-readme

dreispt mentioned this pull request Mar 9, 2024

[15.0][MIG] web_widget_mermaid #2308

Closed

OCA-git-bot added a commit that referenced this pull request Mar 9, 2024

Merge PR #2763 into 15.0

3fb32d3

Signed-off-by dreispt

OCA-git-bot added the bot is merging ⏳ label Mar 9, 2024

OCA-git-bot removed the bot is merging ⏳ label Mar 9, 2024

[REF] pre-commit: copier update

ddea6a9

OCA-git-bot added a commit that referenced this pull request Mar 9, 2024

Merge PR #2763 into 15.0

a851efa

Signed-off-by dreispt

OCA-git-bot added the bot is merging ⏳ label Mar 9, 2024

OCA-git-bot removed the bot is merging ⏳ label Mar 9, 2024

bofiltd approved these changes May 16, 2024

View reviewed changes

OCA-git-bot added a commit that referenced this pull request May 16, 2024

Merge PR #2763 into 15.0

378f77e

Signed-off-by dreispt

OCA-git-bot added the bot is merging ⏳ label May 16, 2024

OCA-git-bot removed the bot is merging ⏳ label May 16, 2024

Merge branch '15.0' into TheRealRoboHeart-15.0-mig-web_widget_mermaid

5b15a17

bosd suggested changes Aug 9, 2025

View reviewed changes

ypapouin mentioned this pull request Jan 26, 2026

[18.0][MIG] web_widget_mermaid: Migration to 18.0 #3419

Open

Uh oh!

Conversation

dreispt commented Mar 9, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dreispt commented Mar 9, 2024

Uh oh!

OCA-git-bot commented Mar 9, 2024

Uh oh!

OCA-git-bot commented Mar 9, 2024

Uh oh!

dreispt commented Mar 9, 2024

Uh oh!

OCA-git-bot commented Mar 9, 2024

Uh oh!

OCA-git-bot commented Mar 9, 2024

Uh oh!

dreispt commented May 16, 2024

Uh oh!

OCA-git-bot commented May 16, 2024

Uh oh!

OCA-git-bot commented May 16, 2024

Uh oh!

idamonli commented May 26, 2024

Uh oh!

bosd commented May 27, 2024

Uh oh!

hbrunn commented Jul 18, 2025

Uh oh!

OCA-git-bot commented Jul 18, 2025

Uh oh!

bosd left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

dreispt commented Mar 9, 2024 •

edited

Loading