8000 Typerror rendering Mermaid diagram · Issue #7815 · outline/outline · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Typerror rendering Mermaid diagram #7815

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
Closed
andymarden opened this issue Oct 21, 2024 · 6 comments · Fixed by #7964
Closed

Typerror rendering Mermaid diagram #7815

andymarden opened this issue Oct 21, 2024 · 6 comments · Fixed by #7964
Assignees
@tommoor
Labels
bug editor more information needed This issue will be automatically closed unless more details are provided

Comments

@andymarden
Copy link
andymarden commented Oct 21, 2024 •
edited
Loading

I have a page with a mermaid diagram embedded. In place of the diagram it says (edit or view mode):

image

If I refresh the page, it then renders OK. Quite a blocker to seamless use! This is on your cloud-hosted implementation

I don't think it is code-dependent (race condition with browser loading?) but here is an example:

flowchart TD
    U([web-gui])
    subgraph "xxxxxx Service"
        R["xxxxxx-rest-api"]
        RDB[(xxxxxxxxxxxxxxx)]
        R <--> RDB
    end
    subgraph "xxxxxxxxxxxx"
        A["xxxxxxxx"]
        E[(processed_events.json)]
        T[Templates.json]
        A --> E 
        T --> A
    end
    subgraph "xxxxxx Service"
        CW["xxxxxx-grpc-api server\n + xxxxxx client"]
        D[("xxxxxr Database (TTTTTTTT)")]
        CW -->|"batch:\ntransfers[1..8090]"| D
        D -->|"xxxxxx[]"| CW

    end
    U -->|"read request"| R
    R -->|"lookup xxxxxx"| CW

    subgraph "performance testing"
        X[transfers]
        G[xxxxxxx.py]
        PT["xxxxxxx.py\n + xxxxxxxx client"]
        G --> X --> PT --> D
    end
@andymarden andymarden added the bug label Oct 21, 2024
@linear Linear
Copy link
linear bot commented Oct 21, 2024

OLN-712 Typerror rendering Mermaid diagram

@tommoor tommoor added the editor label Oct 21, 2024
@tommoor
Copy link
Member
tommoor commented Oct 22, 2024 •
edited
Loading

I can't reproduce, though I have seen this error in our logs – could you provide some more information about the reset of the content that's on the page and in which situations the error appears vs doesnt, your browser, and the other info that's requested in the bug template.

@tommoor tommoor self-assigned this Oct 22, 2024
@tommoor tommoor added the more information needed This issue will be automatically closed unless more details are provided label Oct 22, 2024
tommoor added a commit that referenced this issue Oct 22, 2024
@tommoor
fix: Improve safety around image toolbar, related #7815
c8d307c
@intech
Copy link
intech commented Oct 30, 2024

I suspect that the error occurs due to cached pages. When we refresh a specific page, the diagram renders correctly; however, when we switch to other pages, the same error reappears.

Additionally, there's another scenario where the diagram appears on one page, but an error shows up on another. After refreshing that second page, the diagram appears correctly there. However, if we then return to the first page (where the diagram was visible before), we find that there is now an error there as well. Refreshing the page resolves the issue.

@tommoor
Copy link
Member
tommoor commented Oct 30, 2024

@andymarden are you still able to reproduce?

@tommoor tommoor closed this as completed Nov 6, 2024
@andymarden
Copy link
Author

Sorry - let me try again.

@tommoor tommoor mentioned this issue Nov 16, 2024
Merged
@enginewang
Copy link

I also encounter this problem, the same error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tommoor tommoor

Labels
bug editor more information needed This issue will be automatically closed unless more details are provided
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mermaid v11 outline/outline
5 participants
@tommoor @intech @enginewang @andymarden and others
0