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

Documentation: Update link in CONTRIBUTING.md #94565

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

Gearhartlove
Copy link
Contributor

Hey there, first time contributing and I'm in the middle of reading the contribution best practice, so sorry for any mistakes in this pull request.

When clicking the Contributing section link under the Contributors guidelines title, this linked to the Godot page not found link.

I changed the link from https://docs.godotengine.org/en/latest/contributing/ways_to_contribute.html to https://docs.godotengine.org/en/latest/contributing/how_to_contribute.html.

Now, when a user clicks the link, it will redirect them to the start of the contribution documentation.

@Gearhartlove
Copy link
Contributor Author

It looks like there is more than one spot where the invalid link is used, updating those other spots now.

Copy link

@tomfull123 tomfull123 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@RedMser

This comment was marked as resolved.

Hey there, first time contributing and I'm in the middle of reading the contribution best practice, so sorry for any mistakes in this pull request. 

When clicking the **Contributing section** link under the **Contributors guidelines** title, this linked to the Godot *page not found* link.

I changed the link from `https://docs.godotengine.org/en/latest/contributing/ways_to_contribute.html` to `https://docs.godotengine.org/en/latest/contributing/how_to_contribute.html`.

Now, when a user clicks the link, it will redirect them to the start of the contribution documentation.
@Gearhartlove
Copy link
Contributor Author

Ok should be rebased now 👍

@Calinou
Copy link
Member

Calinou commented Jul 21, 2024

When clicking the Contributing section link under the Contributors guidelines title, this linked to the Godot page not found link.

That means the redirect isn't working correctly, even though it was added in godotengine/godot-docs#9609.

@Calinou Calinou added the bug label Jul 21, 2024
@AThousandShips
Copy link
Member

@Gearhartlove
Copy link
Contributor Author

Thank you @AThousandShips for updating the CSV redirects, I didn't know that existed 😅 .

One question I have is, do we want the "old link" to be in the CONTRIBUTING.md doc still, or does it not matter because we redirect?

@AThousandShips
Copy link
Member

Should be updated here I'd say to help along

@akien-mga akien-mga changed the title Documentation: update link in CONTRIBUTING.md Documentation: Ypdate link in CONTRIBUTING.md Jul 22, 2024
@akien-mga akien-mga changed the title Documentation: Ypdate link in CONTRIBUTING.md Documentation: Update link in CONTRIBUTING.md Jul 22, 2024
@akien-mga akien-mga added this to the 4.3 milestone Jul 22, 2024
@akien-mga akien-mga added documentation cherrypick:3.x Considered for cherry-picking into a future 3.x release cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release and removed cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release labels Jul 22, 2024
@akien-mga akien-mga merged commit 1c71313 into godotengine:master Jul 22, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

@Gearhartlove Gearhartlove deleted the update-contribution-link branch July 22, 2024 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:3.x Considered for cherry-picking into a future 3.x release documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants