sedona icon indicating copy to clipboard operation
sedona copied to clipboard

docs: broken links

Open jbampton opened this issue 9 months ago • 7 comments

Some 404 not found broken links in our docs need to be cleaned up.

[docs/api/flink/Function.md]:
[404] http://www.kr.tuwien.ac.at/staff/eiter/et-archive/cdtr9464.pdf | Network error: Not Found

[docs/api/sql/Function.md]:
[404] http://www.kr.tuwien.ac.at/staff/eiter/et-archive/cdtr9464.pdf | Network error: Not Found

[docs/api/snowflake/vector-data/Function.md]:
[404] http://www.kr.tuwien.ac.at/staff/eiter/et-archive/cdtr9464.pdf | Network error: Not Found

[docs/tutorial/viz.md]:
[404] https://github.com/apache/sedona/tree/master/examples/spark-viz | Network error: Not Found
[404] https://github.com/apache/sedona/tree/master/examples/viz | Network error: Not Found

[docs/tutorial/files/stac-sedona-spark.md]:
[404] https://www.youtube.com/watch?v=stac-video | Network error: Not Found

jbampton avatar Mar 29 '25 00:03 jbampton

@jbampton on it, do we need to clear it out or update to real links?

Brijeshthummar02 avatar Mar 29 '25 06:03 Brijeshthummar02

Hello @Brijeshthummar02 I found these broken links today.

I am not sure what the correct links should be.

It would be best to find the correct current online links.

It is also possible to use The Wayback Machine to find old versions of web pages and files etc that are no longer live for example for the 404 on the PDF file.

https://web.archive.org/

jbampton avatar Mar 29 '25 06:03 jbampton

Also while going through the markdown files i saw tones of lint issues if u want i can handle it, small req would be good if you provide me the above correct links as i am not knowing much about repo.

Brijeshthummar02 avatar Mar 29 '25 06:03 Brijeshthummar02

Let us keep each PR self contained and focused on just one issue.

So if you want to fix Markdown lint issues do that in a second PR.

We already have lots of linters running with pre-commit including markdownlint.

Whilst it is nice to clean up the docs it is better to also add a check / test to enforce these cleans ups in future and to stop regressions.

So the markdownlint config is here: https://github.com/apache/sedona/blob/master/.github/linters/.markdown-lint.yml

The pre-commit config is in the repo root.

jbampton avatar Mar 29 '25 06:03 jbampton

@jiayuasu may i get the correct link so that i can fix it.

Brijeshthummar02 avatar Mar 30 '25 13:03 Brijeshthummar02

Hello @Brijeshthummar02

For the three broken links for the PDF files I have found the correct link here:

https://www.kr.tuwien.ac.at/staff/eiter/et-archive/files/cdtr9464.pdf

Thanks

jbampton avatar Apr 01 '25 02:04 jbampton

I am working on this issue

jjestrada2 avatar Aug 07 '25 02:08 jjestrada2