Skip to content

Conversation

@r00gm
Copy link
Contributor

@r00gm r00gm commented May 28, 2025

Summary

Allow users to change workflow ownership within the editor

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/PAY-2630/add-change-owner-option-into-the-three-dots-menu-when-on-the-workflow

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)

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.

cubic found 2 issues across 3 files. Review them in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

Attention: Patch coverage is 95.83333% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...r-ui/src/components/MainHeader/WorkflowDetails.vue 94.64% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label May 28, 2025
@r00gm r00gm changed the title feat: add change owner to editor feat(editor): Added "Change owner" option to editor May 28, 2025
@r00gm r00gm changed the title feat(editor): Added "Change owner" option to editor feat(editor): Add "Change owner" option to editor May 28, 2025
@r00gm r00gm requested a review from cstuncsik May 29, 2025 06:45
cstuncsik
cstuncsik previously approved these changes May 30, 2025
MarcL
MarcL previously approved these changes Jun 2, 2025
@shortstacked
Copy link
Contributor

Workflow Test Results 📊 🔴 1 Failed, ⚠️ 2 Warnings, 👍 80 Successful out of 83 total workflows.

Detail: Workflows failing: 237: Workflow contains 1 deleted data. View full workflow run

Tested Ref: 97a022abf0b41cdeb10b530e4960a168a53cfe76 by @MarcL

❌ Failed Tests (1)

Workflow ID Workflow Name Reason
237 BasicLLMChain:AzureChat Workflow contains 1 deleted data.

⚠️ Warnings (2)

Workflow ID Workflow Name Reason
35 Slack:User:getPresence info:UserProfile:get update... Workflow contains new data that previously did not exist.
257 Agent:auto-fix:anthropic Workflow contains new data that previously did not exist.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2025

⚠️ Some Cypress E2E specs are failing, please fix them before merging

@r00gm r00gm requested a review from MarcL June 2, 2025 11:39
@shortstacked
Copy link
Contributor

Workflow Test Results 📊 🔴 2 Failed, ⚠️ 3 Warnings, 👍 78 Successful out of 83 total workflows.

Detail: Workflows failing: 237: Workflow contains 1 deleted data. View full workflow run

Tested Ref: f12fb6d9a0539dee4dc47e3a76035983d3326aba by @MarcL

❌ Failed Tests (2)

Workflow ID Workflow Name Reason
237 BasicLLMChain:AzureChat Workflow contains 1 deleted data.
258 Agent:auto-fix:openai Workflow contains 2 deleted data.

⚠️ Warnings (3)

Workflow ID Workflow Name Reason
35 Slack:User:getPresence info:UserProfile:get update... Workflow contains new data that previously did not exist.
53 ConvertKit:CustomField:create getAll update delete... Workflow contains new data that previously did not exist.
257 Agent:auto-fix:anthropic Workflow contains new data that previously did not exist.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2025

✅ All Cypress E2E specs passed

@shortstacked
Copy link
Contributor

Workflow Test Results 📊 🔴 1 Failed, ⚠️ 2 Warnings, 👍 80 Successful out of 83 total workflows.

Detail: Workflows failing: 237: Workflow contains 1 deleted data. View full workflow run

Tested Ref: f12fb6d9a0539dee4dc47e3a76035983d3326aba by @cstuncsik

❌ Failed Tests (1)

Workflow ID Workflow Name Reason
237 BasicLLMChain:AzureChat Workflow contains 1 deleted data.

⚠️ Warnings (2)

Workflow ID Workflow Name Reason
35 Slack:User:getPresence info:UserProfile:get update... Workflow contains new data that previously did not exist.
257 Agent:auto-fix:anthropic Workflow contains new data that previously did not exist.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2025

⚠️ Some Cypress E2E specs are failing, please fix them before merging

@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2025

✅ All Cypress E2E specs passed

@r00gm r00gm merged commit 5bc4e5d into master Jun 3, 2025
21 checks passed
@r00gm r00gm deleted the pay-2630-add-change-owner-option-into-the-three-dots-menu-when-on-the branch June 3, 2025 07:59
Alexandero89 pushed a commit to Alexandero89/n8n that referenced this pull request Jun 4, 2025
Alexandero89 pushed a commit to Alexandero89/n8n that referenced this pull request Jun 4, 2025
@janober
Copy link
Member

janober commented Jun 11, 2025

Got released with n8n@1.98.0

TianYi0217 pushed a commit to TianYi0217/n8n that referenced this pull request Jun 14, 2025
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 Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants