Skip to content

feat: add status to insight project#3917

Open
gaspergrom wants to merge 1 commit intomainfrom
feat/status-project-insights
Open

feat: add status to insight project#3917
gaspergrom wants to merge 1 commit intomainfrom
feat/status-project-insights

Conversation

@gaspergrom
Copy link
Contributor

@gaspergrom gaspergrom commented Mar 13, 2026

Note

Low Risk
Low risk: adds a new status field to the Tinybird project insights datasource and selects it through existing pipes, plus a minor schedule shift for the copy job.

Overview
Adds a status column to project_insights_copy_ds and propagates it through the copy pipeline (project_insights_copy.pipe) and the serving endpoint (project_insights.pipe) so consumers can see each project’s current status.

Adjusts the copy pipe’s COPY_SCHEDULE from 0 2 * * * to 0 3 * * *.

Written by Cursor Bugbot for commit c373dad. This will update automatically on new commits. Configure here.

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI review requested due to automatic review settings March 13, 2026 10:22
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the project status field to the Project Insights dataset served via Tinybird, so consumers can differentiate e.g. active vs archived projects when fetching insights.

Changes:

  • Add status to the materialized project_insights_copy_ds dataset (copy pipe + datasource schema/docs).
  • Expose status in the project_insights.pipe endpoint response.
  • Shift the project_insights_copy copy schedule from 0 2 * * * to 0 3 * * *.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
services/libs/tinybird/pipes/project_insights_copy.pipe Includes status in base + final selects and adjusts the COPY_SCHEDULE.
services/libs/tinybird/pipes/project_insights.pipe Returns status from project_insights_copy_ds in the endpoint query.
services/libs/tinybird/datasources/project_insights_copy_ds.datasource Adds status to schema and documents the new column.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 11 to 14
logoUrl,
isLF,
status,
contributorCount,
Comment on lines 18 to 22
slug,
logoUrl,
isLF,
status,
contributorCount,
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