Skip to content

Rename http to https links and fix broken ones#2967

Merged
Dunbaratu merged 8 commits intoKSP-KOS:developfrom
kna27:rename_links
Jul 13, 2021
Merged

Rename http to https links and fix broken ones#2967
Dunbaratu merged 8 commits intoKSP-KOS:developfrom
kna27:rename_links

Conversation

@kna27
Copy link
Contributor

@kna27 kna27 commented Jun 14, 2021

Fixes #2966

Update links in the kOS error messages in the terminal from HTTP to HTTPS. Additionally fixes broken links that point to nowhere and give a 404 error.

@kna27 kna27 changed the title [Not Ready] Rename http to https links and fix broken ones Rename http to https links and fix broken ones Jun 14, 2021
@kna27
Copy link
Contributor Author

kna27 commented Jun 14, 2021

This is now ready to be merged, however, I am not sure about one of the broken links.
I replaced https://ksp-kos.github.io/KOS_DOC/summary_topics/coordframe/raw/ with https://ksp-kos.github.io/KOS_DOC/math/geocoordinates.html. If someone could confirm if this is right or tell me what to change it to, it would be appreciated. Other than that, ready to merge!

@nuggreat
Copy link

nuggreat commented Jun 14, 2021

depending on the context of https://ksp-kos.github.io/KOS_DOC/summary_topics/coordframe/raw/ it could also be intended to point at https://ksp-kos.github.io/KOS_DOC/math/ref_frame.html#origin-position

EDIT: looking at the surrounding text in your changes it definitely should be referencing the https://ksp-kos.github.io/KOS_DOC/math/ref_frame.html#origin-position not geocoordinates as everything in there has to do with vectors and one of the critical things when it comes to working with vectors is knowing the origin of the vector.

EDIT2: the velocity related suffixes might also be better served by linking to: https://ksp-kos.github.io/KOS_DOC/structures/orbits/orbitablevelocity.html as they do not directly return vectors.

@kna27
Copy link
Contributor Author

kna27 commented Jun 14, 2021

Fixed, thank you!

@Dunbaratu Dunbaratu merged commit 4c5acd7 into KSP-KOS:develop Jul 13, 2021
@kna27 kna27 deleted the rename_links branch July 14, 2021 01:04
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.

http link in terminal should be https

3 participants