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.

Sign up for GitHub

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

Jump to bottom

--> on_call is not escaped? #5134

Closed
mitinarseny opened this issue Dec 11, 2023 · 1 comment
Closed

--> on_call is not escaped? #5134

mitinarseny opened this issue Dec 11, 2023 · 1 comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@mitinarseny
Copy link

mitinarseny commented Dec 11, 2023 •
edited
Loading

Description

Take a look at these two almost identical flowcharts. The first one uses on_call1 node name and it works. While the second one uses on_call and it shows an error. Looks like something is not properly escaped...

flowchart LR
    a --> on_call1
Loading 
flowchart LR
    a --> on_call
Loading

Steps to reproduce

have a flowchart with a link to a node called on_call

Screenshots

image

Setup

  • Mermaid version:
  • Browser and Version: Safari

Additional Context

Meanwhile, both on_call and on_call1 work well in VS Code markdown preview extension. So the bug might be in GitHub's integration for mermaid
@mitinarseny mitinarseny added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Dec 11, 2023
@sidharthv96
Copy link
Member

GitHub has an old version, works well in mermaid.live

info
Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mitinarseny @sidharthv96