Skip to content

Conversation

@nikhilkuria
Copy link
Contributor

Summary

The event Workflow first prod success is emitted when a workflow succeeds for the first time in prod.
However, this was only sent for workflows in a personal project.

This has been changes to include all workflows.
For workflows in personal project

  • project id and user id are sent in the payload

For workflows in other project

  • only project id is sent

This behaviour is documented here

Related Linear tickets, Github issues, and Community forum posts

https://n8nio.slack.com/archives/C04B1GZ4T0U/p1762944806681249

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Nov 24, 2025
@nikhilkuria nikhilkuria changed the title fix: send prod workflow succeeded for wfs in projects fix(core): Send prod workflow succeeded for wfs in projects Nov 24, 2025
@nikhilkuria nikhilkuria requested a review from r00gm November 24, 2025 10:41
@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...es/cli/src/services/workflow-statistics.service.ts 66.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

@currents-bot
Copy link

currents-bot bot commented Nov 24, 2025

E2E Tests: n8n tests timed out after 30m

🟢 75 · 🔴 0 · ⚪️ 0

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Timed out

  • Commit: 7a5d51f

  • Spec files: 96

  • Overall tests: 600

  • Duration: 30m

  • Parallelization: 1

Groups

GroupId Results Spec Files Progress
ui 🟢 75 · 🔴 0 · ⚪️ 0 15 / 90
ui:isolated 🟢 0 · 🔴 0 · ⚪️ 0 0 / 6


This message was posted automatically by currents.dev | Integration Settings

@blacksmith-sh

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants