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

Fix broken links #662

Merged
merged 3 commits into from
Jan 25, 2025
Merged

Fix broken links #662

merged 3 commits into from
Jan 25, 2025

Conversation

FraglyG
Copy link
Contributor

@FraglyG FraglyG commented Jan 23, 2025

  • CLI installation references were didn't have a / prefix
  • Mongo Replica Set template's URL changed to ha-mongo

- CLI installation references were didn't have a `/` prefix
- Mongo Replica Set template's URL changed to ha-mongo
@brody192
Copy link
Contributor

I wonder if the other HA template links are outdated given I did change them all to ha-* after they where made public.

@FraglyG
Copy link
Contributor Author

FraglyG commented Jan 23, 2025

I just checked and couldn't find any other broken HA template links, not sure if you fixed them since you've said that
I did find some other broken links though, I'll add them to this PR later

@FraglyG FraglyG changed the title Mongo Replica Set: Fix broken links Fix broken links Jan 24, 2025
@FraglyG
Copy link
Contributor Author

FraglyG commented Jan 24, 2025

I took the liberty of going through all of the template links I could find (I might've missed some but I covered most of them) to make sure they're all in tact, it seems they all are!

I did notice however that some pages reference the outdated Chroma template, would it be fine if I replaced those links with my Chroma template? It's more up-to-date and maintained.

Pages referencing outdated Chroma template:

I also found 11 https://railway.app references, should I convert these to https://railway.com while I'm at it too?

@brody192
Copy link
Contributor

Yep you are free to replace the template with your template, and update the https://railway.app links!

@FraglyG
Copy link
Contributor Author

FraglyG commented Jan 25, 2025

Awesome! Thank you :D
I believe that should be all then

@brody192 brody192 merged commit 747284b into railwayapp:main Jan 25, 2025
1 check passed
@brody192
Copy link
Contributor

Thank you!

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.

2 participants