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

Link to webhelp #5316

Merged
merged 4 commits into from Feb 26, 2024
Merged

Link to webhelp #5316

merged 4 commits into from Feb 26, 2024

Conversation

BaumiCoder
Copy link
Contributor

@BaumiCoder BaumiCoder commented Feb 25, 2024

📑 Summary

There are still some old links to the markdown files or the mermaid-js.github.io site. I replaced them with the corresponding links to mermaid.js.org. (Similar changes like in PR mermaid-js/mermaid-live-editor#1405)

📏 Design Decisions

📋 Tasks

Make sure you

The constributing page is not a plain markdown any more.
It is part of the generated webhelp, so link to this. Some of the links
were still to the symbolic link at the root of the repository.
The docs are availabe as webhelp, which is a nicer viewer than
the plain markdown at GitHub. So a little search & replace:
- "(./docs" --> "https://mermaid.js.org"
  vim command ":%s/(.\/docs/(https:\/\/mermaid.js.org/g"
- ".md)" --> ".html)"
  vim command ":%s/.md)/.html)/g"
Replace each link to mermaid-js.github.io with corresponding link
at the webhelp (/ website), using the redirects to get these links.
(The redirects are to the http:// sites. I replace it with the https://
links of course.)
Copy link

netlify bot commented Feb 25, 2024

Deploy Preview for mermaid-js failed.

Name Link
🔨 Latest commit e371d72
🔍 Latest deploy log https://app.netlify.com/sites/mermaid-js/deploys/65dcadd78d62400008925fe6

Copy link

codecov bot commented Feb 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.15%. Comparing base (70c8a3d) to head (e79a08e).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #5316   +/-   ##
========================================
  Coverage    79.15%   79.15%           
========================================
  Files          175      175           
  Lines        14514    14514           
  Branches       866      866           
========================================
  Hits         11488    11488           
  Misses        2823     2823           
  Partials       203      203           
Flag Coverage Δ
e2e 84.72% <ø> (ø)
unit 43.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@sidharthv96 sidharthv96 merged commit 4e12157 into mermaid-js:develop Feb 26, 2024
8 of 10 checks passed
Copy link

mermaid-bot bot commented Feb 26, 2024

@BaumiCoder, Thank you for the contribution!
You are now eligible for a year of Premium account on MermaidChart.
Sign up with your GitHub account to activate.

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.

None yet

2 participants