Skip to content

Non Resolving Link Target in README.md #229

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

Closed
chris-bishop opened this issue Aug 1, 2021 · 2 comments
Closed

Non Resolving Link Target in README.md #229

chris-bishop opened this issue Aug 1, 2021 · 2 comments

Comments

@chris-bishop
Copy link

chris-bishop commented Aug 1, 2021

Hello!

Firstly, let me thank NASA for these outstanding contributions to the Open Source Community, and of course for continuing to push humanity forward through innovation and exploration!

I was referring back to the README.md file to nav my way back in to some example code and reference material today, and happened to discover the following markdown link ref basically == (Request) --> /dev/null to "black hole":

https://www.github.com./NASAWorldWind/WorldWindJava/tree/develop/README.md#L31

- [Java Demos](https://goworldwind.org/demos) has a complete list of example apps

I'm not sure if the site below was in fact the original intended target and has since migrated under a new DNS - however, in either case, given the context it seems they may make a worthy replacement 😄

https://live.osgeo.org/archive/10.0/en/quickstart/worldwindjava_quickstart.html

If I wasn't currently traveling and stuck with my phone only I would submit a PR to at least remove the non resolving endpoint, so I at least wanted to backlog an Issue.

When I finally return home if someone hasn't beat me to it I will do so for sure - would be the least I could do given all NASA does for the world!

To infinity and beyond!

Cheers,

@chris-bishop - Git
Chris Bishop - LinkedIn

@Beak-man
Copy link
Member

Hello @chris-bishop. We fixed that outdated reference in the README a while ago but indeed we had some other hyperlinks in the project with references to the old website. PR #235 is submitted to address this. Thank you for the heads-up and the warm comments!

@chris-bishop
Copy link
Author

chris-bishop commented Jan 29, 2022

Hey @Beak-man!

Apologies for only JUST now acknowledging your comment and PR #235!

And that's a big "Affirmative Ground Control!" (please excuse my overuse of Space puns - I have a pun sickness!)

README.md links now all work as expected from the fresh clone of the develop branch I just took locally - thank you for taking the time to fix this - and for everything you and the team at NASA do for us!

👍🏼 💯 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants