Skip to content

Fix: mermaid diagrams with parameters in site config#1316

Merged
geriom merged 1 commit intogoogle:mainfrom
deining:mermaid-regression
Dec 1, 2022
Merged

Fix: mermaid diagrams with parameters in site config#1316
geriom merged 1 commit intogoogle:mainfrom
deining:mermaid-regression

Conversation

@deining
Copy link
Contributor

@deining deining commented Nov 23, 2022

This PR fixes this two regressions introduced with 90843e3:

  • build breaks if mermaid diagram(s) are displayed on one or more pages and parameters for mermaid are given in the site config.
  • parameters for mermaid given in the site config were picked up only if mermaid.enable=true was given. With auto-activation of mermaid diagrams in place, parameters should be considered event if mermaid.enable is missing in the site config.

This PR also bumps mermaid to the latest released version 9.2.2.

@deining deining added the regression Feature that has worked before stopped working label Nov 23, 2022
@deining deining requested a review from geriom November 23, 2022 07:54
Copy link
Contributor

@multun multun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm this patch fixes the regression 🎉

Copy link
Collaborator

@geriom geriom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and can confirm this works. Thank you!

@geriom geriom merged commit 33f3dfe into google:main Dec 1, 2022
@deining deining deleted the mermaid-regression branch December 1, 2022 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

regression Feature that has worked before stopped working

4 participants