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

Branch tracking badge links have broken links #857

Closed
rparrett opened this issue Jan 9, 2024 · 2 comments · Fixed by #861
Closed

Branch tracking badge links have broken links #857

rparrett opened this issue Jan 9, 2024 · 2 comments · Fixed by #861
Labels
A-Book C-Webdev E-Good-First-Issue Good for newcomers

Comments

@rparrett
Copy link
Contributor

rparrett commented Jan 9, 2024

bevy-website/content/learn/book/plugin-development/_index.md

Lines 99 to 100 in 0320341

|[![Bevy tracking](https://img.shields.io/badge/Bevy%20tracking-main-lightblue)](https://github.com/bevyengine/bevy/blob/main/docs/plugins_guidelines.md#main-branch-tracking)|I intend to track main as much as I can|`[![Bevy tracking](https://img.shields.io/badge/Bevy%20tracking-main-lightblue)](https://github.com/bevyengine/bevy/blob/main/docs/plugins_guidelines.md#main-branch-tracking)`|
|[![Bevy tracking](https://img.shields.io/badge/Bevy%20tracking-released%20version-lightblue)](https://github.com/bevyengine/bevy/blob/main/docs/plugins_guidelines.md#main-branch-tracking)|I will only follow released Bevy versions|`[![Bevy tracking](https://img.shields.io/badge/Bevy%20tracking-released%20version-lightblue)](https://github.com/bevyengine/bevy/blob/main/docs/plugins_guidelines.md#main-branch-tracking)`|

They link to https://github.com/bevyengine/bevy/blob/main/docs/plugins_guidelines.md#main-branch-tracking which no longer exists after bevyengine/bevy#11242.
@rparrett
Copy link
Contributor Author

rparrett commented Jan 9, 2024 •
edited
Loading

We will probably need to add insert_anchor_links = "right" so that we can have an equivalent link in the new document.

@rparrett rparrett mentioned this issue Jan 9, 2024
Closed
@rparrett
Copy link
Contributor Author

Another option might be to remove the link entirely in favor of alt text.

@rparrett rparrett mentioned this issue Jan 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Book C-Webdev E-Good-First-Issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Plugin guidelines makeover rparrett/bevy-website
2 participants
@rparrett @alice-i-cecile