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

add twenty-tinybird package #8030

Merged
merged 11 commits into from
Oct 29, 2024
Merged

add twenty-tinybird package #8030

merged 11 commits into from
Oct 29, 2024

Conversation

anamarn
Copy link
Contributor

@anamarn anamarn commented Oct 24, 2024

https://github.com/twentyhq/private-issues/issues/75

TLDR

Add twenty-server package, wich contains the last tinybird datasources and pipes used in analytics.

This new version of the API endpoints, pipes and datasources are inspired by the implementation of dub.co (https://github.com/dubinc/dub/blob/main/packages/tinybird/).

It contains the webhooks analytics, serverless functions duration and serverless functions error count analytics. As well as

In order to test

  • Follow the instructions in the README.md on twenty-tinybird using your admin token from twenty_analytics_playground if you want to modify them.
  • For a better experience add the extension Tinybird support for VsCode
  • If you want more info about datasources and pipes please read the tinybird docs.

Create a virtual enviroment and install tinybird
```sh
python3 -m venv .venv
source .venv/bin/activate
Copy link
Member

Choose a reason for hiding this comment

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

add .venv to .gitignore then :)

[5] eu-central-1 (aws) (https://app.tinybird.co/aws/eu-central-1) <- this
[0] Cancel

Use region [1]:
Copy link
Member

Choose a reason for hiding this comment

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

Region 5 no?

```
Things I learned:

When creating a MV think about populating it first using the files from the fixtures
Copy link
Member

Choose a reason for hiding this comment

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

MV?

Copy link
Member

Choose a reason for hiding this comment

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

add bullet points for lisibility?

@@ -0,0 +1,21 @@

Copy link
Member

Choose a reason for hiding this comment

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

useless line

@@ -0,0 +1,58 @@

Copy link
Member

Choose a reason for hiding this comment

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

same here

@FelixMalfait FelixMalfait merged commit 8ef4efa into main Oct 29, 2024
18 checks passed
@FelixMalfait FelixMalfait deleted the feat/twenty-tinybird branch October 29, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants