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

feat: migrate leafwatch to use clickhouse client #3964

Merged
merged 1 commit into from Nov 6, 2023
Merged

Conversation

bigint
Copy link
Member

@bigint bigint commented Nov 6, 2023

What does this PR do?

πŸ€– Generated by Copilot at 440f676

This pull request refactors the profile details functionality of the hey app. It moves the logic of fetching profile details from the leafwatch worker to the stats worker, and uses the @hey/clickhouse package to connect to and query the ClickHouse database. It also updates the web component and the environment variables to reflect the changes.

Related issues

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (non-breaking small changes to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Explanation of the changes

πŸ€– Generated by Copilot at 440f676

Emoji

πŸ”„πŸš€πŸ—‘οΈ

Copy link

vercel bot commented Nov 6, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Updated (UTC)
prerender βœ… Ready (Inspect) Visit Preview Nov 6, 2023 6:31am
web βœ… Ready (Inspect) Visit Preview Nov 6, 2023 6:31am

Copy link

height bot commented Nov 6, 2023

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

πŸ’‘Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@bigint bigint enabled auto-merge November 6, 2023 06:26
@bigint bigint disabled auto-merge November 6, 2023 06:27
@bigint bigint merged commit 80a970d into main Nov 6, 2023
6 of 8 checks passed
@bigint bigint deleted the lf-ch-client branch November 6, 2023 06:27
@vercel vercel bot temporarily deployed to Preview – prerender November 6, 2023 06:27 Inactive
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.

None yet

1 participant