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 links to jonatas.github.io/timescaledb #99

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

MasterOdin
Copy link
Contributor

@MasterOdin MasterOdin commented Jan 16, 2025

PR fixes a couple of links I found to jonatas.github.io/timescaledb to work with the new repo location. For the three images, I chose to use relative paths as mkdocs seemed to be fine with it locally, and this seemed less brittle in case the repo ever changed names again.

@MasterOdin MasterOdin force-pushed the mpeveler/fix-docs-images branch from 53321c8 to 97f91c3 Compare January 16, 2025 00:59
@@ -1,5 +1,5 @@
# ruby candlestick.rb postgres://user:pass@host:port/db_name
# @see https://jonatas.github.io/timescaledb/candlestick_tutorial
# @see https://timescale.github.io/timescaledb-ruby/toolkit_candlestick/
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasn't sure if this was correct, as I'm not sure there ever was a documented called candlestick_tutorial in the repo. 🤷

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes! it's correct!

Thank you very much! I removed almost all references, but some of the refs in the code are still there!

@@ -23,7 +23,7 @@ In this example, we're going to use the [lttb][3] function, that is part of the
The image bellow corresponds to the step by step guide provided here.


![LTTB Zoomable Example](https://jonatas.github.io/timescaledb/img/lttb_zoom.gif)
![LTTB Zoomable Example](img/lttb_zoom.gif)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Very good! I remember I faced some issues using relative links but let's try again.

Copy link
Collaborator

@jonatas jonatas left a comment

Choose a reason for hiding this comment

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

Thanks @MasterOdin!

@jonatas jonatas merged commit 882517f into timescale:master Jan 16, 2025
4 checks passed
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