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

Documentation for visualizing ClickHouse data in Hashboard #1539

Merged
merged 5 commits into from
Sep 28, 2023

Conversation

dse0
Copy link
Contributor

@dse0 dse0 commented Sep 22, 2023

No description provided.

@dse0 dse0 requested a review from a team as a code owner September 22, 2023 19:11
@dse0 dse0 requested review from Paultagoras and removed request for a team September 22, 2023 19:11
@CLAassistant
Copy link

CLAassistant commented Sep 22, 2023

CLA assistant check
All committers have signed the CLA.

@@ -39,6 +39,7 @@ Now that your data is in ClickHouse, it's time to analyze it, which often involv
| [Deepnote](./data-visualization/deepnote.md) | Native connector | ✅ | ✅ | |
| [Explo](./data-visualization/explo-and-clickhouse.md) | Native connector | ✅ | ✅ | |
| [Grafana](./data-visualization/grafana-and-clickhouse.md) | ClickHouse official connector | ✅ | ✅ | |
| [Hashboard](./data-visualization/hashboard-and-clickhouse.md) | Native connector | ✅ | ✅ | |
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done 👍

## Steps to Connect Hashboard to ClickHouse

### 1. Gather Your Connection Details

Copy link
Member

Choose a reason for hiding this comment

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

Could you add import ConnectionDetails from '@site/docs/en/_snippets/_gather_your_details_native.md'; and <ConnectionDetails /> snippet? See an example https://raw.githubusercontent.com/ClickHouse/clickhouse-docs/main/docs/en/integrations/data-visualization/grafana-and-clickhouse.md

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done 👍

@robot-clickhouse-ci-1
Copy link

robot-clickhouse-ci-1 commented Sep 26, 2023

This is an automated comment for commit 2b76d50 with description of existing statuses. It's updated for the latest CI running

✅ Click here to open a full report in a separate page

Successful checks
Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR✅ success
Docs CheckBuilds and tests the documentation✅ success
Mergeable CheckChecks if all other necessary checks are successful✅ success
VercelThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success

@mshustov
Copy link
Member

@justindeguzman for some reason, I don't have access to Docs check logs. Could you provide more details on the CI check failure?

@justindeguzman
Copy link
Contributor

Just upgraded role @mshustov, the deploys should go through now

@mshustov
Copy link
Member

@justindeguzman can we relax the requirements for the stale merges? I can't merge the PR since This branch is out-of-date with the base branch. We don't write rights to the forks (it should be allowed explicitly).
Asking a contributor to constantly merge the main branch or provide ClickHouse team write access to the fork, highers the contribution bar

@dse0
Copy link
Contributor Author

dse0 commented Sep 28, 2023

I went ahead and merged main again, looks like the PR needs re-approval? @mshustov I gave you write access to the fork if it's helpful.

@mshustov mshustov merged commit 8723b37 into ClickHouse:main Sep 28, 2023
2 checks passed
@mshustov mshustov deleted the hashboard-documentation branch September 28, 2023 15:03
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.

5 participants