github-starred/n8n
2
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-12-05 19:27:26 -06:00
Code Issues 1.3k Packages Projects Releases 100 Wiki Activity

[PR #22038] Ado 4295 fb #22005

New Issue
Open
opened 2025-11-20 06:22:29 -06:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-20 06:22:29 -06:00
Owner
Copy Link

📋 Pull Request Information

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

Base: ado-4348-add-new-internal-endpoints ← Head: ADO-4295_fb

📝 Commits (10+)

  • 0c8ffa9 Drop active flag on workflow entity
  • ad47e3b Update tests
  • 6cb77c0 Update more cli code to use activeVersionId
  • 9b1299d Use activeVersionId for IWorkflowBase in workflow tests
  • bfe852f Escape column for license metric query
  • ef1aac8 Improve workflow create test
  • 857a214 Fix Playwright types
  • 2b28c81 Fix state in editor to use activeVersionId
  • b7821b1 Fix frontend unit tests
  • 3b45ae1 Improve test expectations

📊 Changes

128 files changed (+1607 additions, -609 deletions)

View changed files

📝 packages/@n8n/api-types/src/push/workflow.ts (+1 -0)
📝 packages/@n8n/backend-test-utils/src/db/workflows.ts (+35 -12)
📝 packages/@n8n/db/src/entities/execution-data.ts (+2 -13)
📝 packages/@n8n/db/src/entities/index.ts (+3 -0)
📝 packages/@n8n/db/src/entities/types-db.ts (+32 -6)
📝 packages/@n8n/db/src/entities/workflow-entity.ts (+1 -4)
➕ packages/@n8n/db/src/entities/workflow-publish-history.ts (+41 -0)
📝 packages/@n8n/db/src/migrations/common/1763047800000-AddActiveVersionIdColumn.ts (+24 -2)
➕ packages/@n8n/db/src/migrations/common/1763387043735-CreateWorkflowPublishHistoryTable.ts (+48 -0)
📝 packages/@n8n/db/src/migrations/postgresdb/index.ts (+2 -0)
📝 packages/@n8n/db/src/migrations/sqlite/index.ts (+2 -0)
📝 packages/@n8n/db/src/repositories/execution.repository.ts (+6 -4)
📝 packages/@n8n/db/src/repositories/index.ts (+1 -0)
📝 packages/@n8n/db/src/repositories/license-metrics.repository.ts (+1 -1)
➕ packages/@n8n/db/src/repositories/workflow-publish-history.repository.ts (+27 -0)
📝 packages/@n8n/db/src/repositories/workflow-statistics.repository.ts (+9 -2)
📝 packages/@n8n/db/src/repositories/workflow.repository.ts (+28 -14)
➕ packages/@n8n/db/src/utils/normalize-workflow-data.ts (+16 -0)
📝 packages/cli/src/__tests__/active-executions.test.ts (+1 -1)
📝 packages/cli/src/__tests__/active-workflow-manager.test.ts (+4 -2)

...and 80 more files

📄 Description

Summary

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)

🔄 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/22038 **Author:** [@CharlieKolb](https://github.com/CharlieKolb) **Created:** 11/19/2025 **Status:** 🔄 Open **Base:** `ado-4348-add-new-internal-endpoints` ← **Head:** `ADO-4295_fb` --- ### 📝 Commits (10+) - [`0c8ffa9`](https://github.com/n8n-io/n8n/commit/0c8ffa916fbfef72c71929d8d9063d2873acaed7) Drop active flag on workflow entity - [`ad47e3b`](https://github.com/n8n-io/n8n/commit/ad47e3be350f8212cf4b84004080a2275c19915a) Update tests - [`6cb77c0`](https://github.com/n8n-io/n8n/commit/6cb77c00d84dca347a27d845ef043d0fc436486f) Update more cli code to use activeVersionId - [`9b1299d`](https://github.com/n8n-io/n8n/commit/9b1299d7fca2b88b814c363148852d567fa07618) Use activeVersionId for IWorkflowBase in workflow tests - [`bfe852f`](https://github.com/n8n-io/n8n/commit/bfe852f292483fee39a6a6b43b30b1bbc5072e30) Escape column for license metric query - [`ef1aac8`](https://github.com/n8n-io/n8n/commit/ef1aac8ec5db3da40b443c9dc786eb47c4066b5f) Improve workflow create test - [`857a214`](https://github.com/n8n-io/n8n/commit/857a2146751779646d4eb2161999c118dedfe746) Fix Playwright types - [`2b28c81`](https://github.com/n8n-io/n8n/commit/2b28c8144e2a989edf9fbe3d76b9254e8ef96dbc) Fix state in editor to use activeVersionId - [`b7821b1`](https://github.com/n8n-io/n8n/commit/b7821b154c3e6f32bd6760339a099cf2fb5a531d) Fix frontend unit tests - [`3b45ae1`](https://github.com/n8n-io/n8n/commit/3b45ae172ab3a75b1b0681b8d2e4439994454101) Improve test expectations ### 📊 Changes **128 files changed** (+1607 additions, -609 deletions) <details> <summary>View changed files</summary> 📝 `packages/@n8n/api-types/src/push/workflow.ts` (+1 -0) 📝 `packages/@n8n/backend-test-utils/src/db/workflows.ts` (+35 -12) 📝 `packages/@n8n/db/src/entities/execution-data.ts` (+2 -13) 📝 `packages/@n8n/db/src/entities/index.ts` (+3 -0) 📝 `packages/@n8n/db/src/entities/types-db.ts` (+32 -6) 📝 `packages/@n8n/db/src/entities/workflow-entity.ts` (+1 -4) ➕ `packages/@n8n/db/src/entities/workflow-publish-history.ts` (+41 -0) 📝 `packages/@n8n/db/src/migrations/common/1763047800000-AddActiveVersionIdColumn.ts` (+24 -2) ➕ `packages/@n8n/db/src/migrations/common/1763387043735-CreateWorkflowPublishHistoryTable.ts` (+48 -0) 📝 `packages/@n8n/db/src/migrations/postgresdb/index.ts` (+2 -0) 📝 `packages/@n8n/db/src/migrations/sqlite/index.ts` (+2 -0) 📝 `packages/@n8n/db/src/repositories/execution.repository.ts` (+6 -4) 📝 `packages/@n8n/db/src/repositories/index.ts` (+1 -0) 📝 `packages/@n8n/db/src/repositories/license-metrics.repository.ts` (+1 -1) ➕ `packages/@n8n/db/src/repositories/workflow-publish-history.repository.ts` (+27 -0) 📝 `packages/@n8n/db/src/repositories/workflow-statistics.repository.ts` (+9 -2) 📝 `packages/@n8n/db/src/repositories/workflow.repository.ts` (+28 -14) ➕ `packages/@n8n/db/src/utils/normalize-workflow-data.ts` (+16 -0) 📝 `packages/cli/src/__tests__/active-executions.test.ts` (+1 -1) 📝 `packages/cli/src/__tests__/active-workflow-manager.test.ts` (+4 -2) _...and 80 more files_ </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:29 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Backlog
bug-report
Bump
Bumped
closed:duplicate
closed:enhancement/feature
closed:incomplete-template
closed:info
closed:non-english
closed:support-issue
closed:working-as-expected
Duplicate
in linear
In Review
n8n team
Needs Feedback
Needs Feedback:Unable to reproduce
node/improvement
node/issue
node/new
pull-request
Mirrored from GitHub Pull Request
Released
Stale
status:complete
status:in-linear
status:pending-assignment
status:team-assigned
team:adore
team:ai
team:cats
team:nodes
team:payday
triage:complete
triage:in-progress
triage:needs-info
triage:needs-reproduction
triage:pending
triage:stalled
ui
Upcoming Release
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/n8n#22005
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?