Skip to content

Conversation

mutdmour
Copy link
Contributor

@mutdmour mutdmour commented Sep 1, 2025

Summary

Remove "User added action" telemetry event because it's duplicated by "user added node to workflow canvas"

Related Linear tickets, Github issues, and Community forum posts

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 the n8n team Authored by the n8n team label Sep 1, 2025
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 3 files

Copy link

bundlemon bot commented Sep 1, 2025

BundleMon

Unchanged files (2)
Status Path Size Limits
WASM Dependencies
tree-sitter-bash.wasm
181.26KB -
WASM Dependencies
tree-sitter.wasm
74.47KB -

No change in files bundle size

Groups updated (2)
Status Path Size Limits
**/*.js
5.73MB (+367.45KB +6.68%) -
**/*.css
195.22KB (+3.97KB +2.08%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

Copy link

codecov bot commented Sep 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mutdmour mutdmour requested review from a team and CharlieKolb September 2, 2025 07:59
Copy link
Contributor

@CharlieKolb CharlieKolb left a comment

Choose a reason for hiding this comment

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

Can you document in the PR description why we're removing the event? Just for future reference 🙏

@mutdmour
Copy link
Contributor Author

mutdmour commented Sep 2, 2025

thanks @CharlieKolb updated description

Copy link

currents-bot bot commented Sep 2, 2025

E2E Tests: n8n tests passed after 4m 32.2s

🟢 550 · 🔴 0 · ⚪️ 0

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: 50bf6a1

  • Spec files: 112

  • Overall tests: 550

  • Duration: 4m 32.2s

  • Parallelization: 3

Groups

GroupId Results Spec Files Progress
ui 🟢 145 · 🔴 0 · ⚪️ 0 34 / 34
No name 🟢 405 · 🔴 0 · ⚪️ 0 78 / 78


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

@mutdmour mutdmour merged commit a1504cf into master Sep 2, 2025
34 checks passed
@mutdmour mutdmour deleted the ai-1315 branch September 2, 2025 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants