Skip to content

feat: activityRelations_data_copilot pipe#3463

Merged
epipav merged 7 commits intomainfrom
feature/activityRelations-data-copilot-pipe
Oct 9, 2025
Merged

feat: activityRelations_data_copilot pipe#3463
epipav merged 7 commits intomainfrom
feature/activityRelations-data-copilot-pipe

Conversation

@epipav
Copy link
Copy Markdown
Collaborator

@epipav epipav commented Oct 6, 2025

Changes proposed ✍️

Adds a new activityRelations_data_copilot pipe for data copilot.

It gets its data from activityRelations_deduplicated_cleaned_ds. The only difference from the raw data source is that fields that contain PII (username and objectMemberUsername) are omitted from the result set. Agents will use this pipe instead of the raw data source when they need raw data.

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@epipav epipav requested a review from borfast as a code owner October 6, 2025 07:45
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Conventional Commits FTW!

@epipav epipav changed the title feat: activityRelations_data_copilot pipe feat: activityRelations_data_copilot pipe Oct 6, 2025
@epipav epipav self-assigned this Oct 6, 2025
@epipav epipav added the Feature Created by Linear-GitHub Sync label Oct 6, 2025
@epipav epipav requested a review from themarolt as a code owner October 6, 2025 08:03
Comment thread services/libs/tinybird/pipes/activityRelations_data_copilot.pipe Outdated
Copy link
Copy Markdown
Contributor

@borfast borfast left a comment

Choose a reason for hiding this comment

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

I noticed you're getting the isContribution column but I don't know how or if is being used by whoever consumes this pipe. We're going to be removing this column soon, so if it's being used, we need to have a plan for how to change this.

Also, the entire tinybird-ci.yml file is commented out but there's no mention of that in the PR description, so I'm wondering if this was by mistake.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 9, 2025

⚠️ 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
Copy Markdown
Contributor

github-actions Bot commented Oct 9, 2025

⚠️ 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.

epipav added 2 commits October 9, 2025 13:43
…com:CrowdDotDev/crowd.dev into feature/activityRelations-data-copilot-pipe
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 9, 2025

⚠️ 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.

@epipav epipav merged commit c349df1 into main Oct 9, 2025
15 checks passed
@epipav epipav deleted the feature/activityRelations-data-copilot-pipe branch October 9, 2025 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants