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

Many broken internal link fixes. #863

Merged
merged 25 commits into from
Sep 28, 2021
Merged

Conversation

dannon
Copy link
Member

@dannon dannon commented Sep 25, 2021

Some targeted, some global, addressing stuff identified in #860

Also converted a bunch of the older /src/<whatever>/index.md paths to just /<whatever>/ in the process.

Local links status after this:

📝Link summary of 1571 pages checked

🔍 Checked ✅ Successful 🚫 Errors
49723 49550 173

@dannon dannon changed the title Misc link fixes. Many broken internal link fixes. Sep 26, 2021
@dannon dannon marked this pull request as ready for review September 26, 2021 23:38
@dannon dannon requested a review from NickSto September 27, 2021 16:11
@NickSto NickSto added the dependencies Pull requests that update a dependency file label Sep 28, 2021
Copy link
Member

@NickSto NickSto left a comment

Choose a reason for hiding this comment

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

I built and tested it locally, and did spot checks on each commit. Looks like it all works to me. Three thumbs up!

src/build/run.mjs Outdated Show resolved Hide resolved
content/events/2021-01-sv-with-ont/index.md Show resolved Hide resolved
@dannon
Copy link
Member Author

dannon commented Sep 28, 2021

@NickSto I'll merge upstream back in here and add svg and check for other artifacts we'd want to directly link.

@dannon
Copy link
Member Author

dannon commented Sep 28, 2021

@NickSto Thanks for checking through it -- it should be good to go now.

@dannon dannon merged commit 11de326 into galaxyproject:master Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants