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

Update index.md #3419

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from
Open

Conversation

brandon-braner
Copy link

Updated link for PopSql to be an actual link

Description

I noticed the link in the docs to popsql on this page was not correct. Fixed it.

Links

Fixes #[insert issue link, if any]

Writing help

For information about style and word usage, see the style guide

Review checklists

Reviewers: use this section to ensure you have checked everything before approving this PR:

Subject matter expert (SME) review checklist

  • Is the content technically accurate?
  • Is the content complete?
  • Is the content presented in a logical order?
  • Does the content use appropriate names for features and products?
  • Does the content provide relevant links to further information?

Documentation team review checklist

  • Is the content free from typos?
  • Does the content use plain English?
  • Does the content contain clear sections for concepts, tasks, and references?
  • Have any images been uploaded to the correct location, and are resolvable?
  • If the page index was updated, are redirects required
    and have they been implemented?
  • Have you checked the built version of this content?

Updated link for PopSql to be an actual link

Signed-off-by: Brandon Braner <brandon.braner@gmail.com>
@CLAassistant
Copy link

CLAassistant commented Sep 9, 2024

CLA assistant check
All committers have signed the CLA.

@@ -9,7 +9,7 @@ keywords: [queries]

Timescale hypertables _are_ PostgreSQL tables. That means you can query them
with standard SQL commands. In addition to the extra benefits you get from Timescale architecture and query planning,
use [PopSQL][popsql] to work on data with centralized SQL queries, interactive visuals and real-time collaboration
use [PopSQL](https://popsql.com/) to work on data with centralized SQL queries, interactive visuals and real-time collaboration
Copy link
Contributor

Choose a reason for hiding this comment

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

It will be more helpful to link to https://docs.timescale.com/getting-started/latest/run-queries-from-console/, so could you please:

  • revert the change to the link structure, so [PopSQL][popsql] (this is a reference-style link and this is proper formatting)
  • add the following at the very bottom of the page, under [skipscan]:
    [popsql]: /getting-started/:currentVersion:/run-queries-from-console/

Thanks!

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.

3 participants