[PR #22052] chore(core): Add instanceType condition to load insights module only for main aand webhook #22014

Open
opened 2025-11-20 06:22:41 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/n8n-io/n8n/pull/22052
Author: @guillaumejacquart
Created: 11/19/2025
Status: 🔄 Open

Base: masterHead: pay-4121-conditionally-load-modules-depending-on-the-instance-type


📝 Commits (1)

  • 937b2bd Add instanceType condition to load insights module only for main and webhooks

📊 Changes

2 files changed (+19 additions, -9 deletions)

View changed files

📝 packages/cli/src/modules/insights/__tests__/insights.module.test.ts (+14 -1)
📝 packages/cli/src/modules/insights/insights.module.ts (+5 -8)

📄 Description

Summary

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)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/n8n-io/n8n/pull/22052 **Author:** [@guillaumejacquart](https://github.com/guillaumejacquart) **Created:** 11/19/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `pay-4121-conditionally-load-modules-depending-on-the-instance-type` --- ### 📝 Commits (1) - [`937b2bd`](https://github.com/n8n-io/n8n/commit/937b2bd7ab48aea03c4a79e1504fa741b46c19f7) Add instanceType condition to load insights module only for main and webhooks ### 📊 Changes **2 files changed** (+19 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `packages/cli/src/modules/insights/__tests__/insights.module.test.ts` (+14 -1) 📝 `packages/cli/src/modules/insights/insights.module.ts` (+5 -8) </details> ### 📄 Description ## Summary <!-- Describe what the PR does and how to test. Photos and videos are recommended. --> ## Related Linear tickets, Github issues, and Community forum posts <!-- Include links to **Linear ticket** or Github issue or Community forum post. Important in order to close *automatically* and provide context to reviewers. https://linear.app/n8n/issue/ --> <!-- Use "closes #<issue-number>", "fixes #<issue-number>", or "resolves #<issue-number>" to automatically close issues when the PR is merged. --> ## Review / Merge checklist - [ ] PR title and summary are descriptive. ([conventions](../blob/master/.github/pull_request_title_conventions.md)) <!-- **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** --> - [ ] [Docs updated](https://github.com/n8n-io/n8n-docs) or follow-up ticket created. - [ ] Tests included. <!-- A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests. --> - [ ] PR Labeled with `release/backport` (if the PR is an urgent fix that needs to be backported) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the
pull-request
label 2025-11-20 06:22:41 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/n8n#22014
No description provided.