-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Updating terminology to reflect publishing and unpublishing of workflows #3973
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 issues found across 29 files
Prompt for AI agents (all 3 issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="docs/try-it-out/tutorial-first-workflow.md">
<violation number="1" location="docs/try-it-out/tutorial-first-workflow.md:141">
P3: Replace the ungrammatical phrasing “by click” with “by clicking” (or “by selecting”) to keep the instructions clear and consistent with prior steps.</violation>
</file>
<file name="docs/integrations/builtin/core-nodes/n8n-nodes-base.scheduletrigger/common-issues.md">
<violation number="1" location="docs/integrations/builtin/core-nodes/n8n-nodes-base.scheduletrigger/common-issues.md:43">
P3: Fix the typo “worfklow” → “workflow” in the updated documentation sentence.</violation>
</file>
<file name="docs/integrations/builtin/core-nodes/n8n-nodes-base.n8n.md">
<violation number="1" location="docs/integrations/builtin/core-nodes/n8n-nodes-base.n8n.md:36">
P2: The operations list now advertises “Publish a workflow” but the linked section still documents the older “Activate” operation, so the documentation is inconsistent and misleading. Update the target section (and anchor) to describe publishing/unpublishing or revert this entry until the rest of the doc is updated.</violation>
</file>
Since this is your first cubic review, here's how it works:
- cubic automatically reviews your code and comments on bugs and improvements
- Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
- Ask questions if you need clarification on any suggestion
Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR
docs/integrations/builtin/core-nodes/n8n-nodes-base.scheduletrigger/common-issues.md
Outdated
Show resolved
Hide resolved
| * [**Delete** an execution](#delete-execution) | ||
| * Workflow | ||
| * [**Activate** a workflow](#activate-deactivate-delete-and-get-workflow) | ||
| * [**Publish** a workflow](#activate-deactivate-delete-and-get-workflow) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: The operations list now advertises “Publish a workflow” but the linked section still documents the older “Activate” operation, so the documentation is inconsistent and misleading. Update the target section (and anchor) to describe publishing/unpublishing or revert this entry until the rest of the doc is updated.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At docs/integrations/builtin/core-nodes/n8n-nodes-base.n8n.md, line 36:
<comment>The operations list now advertises “Publish a workflow” but the linked section still documents the older “Activate” operation, so the documentation is inconsistent and misleading. Update the target section (and anchor) to describe publishing/unpublishing or revert this entry until the rest of the doc is updated.</comment>
<file context>
@@ -33,7 +33,7 @@ The HTTP Request node has options to [provide the SSL certificate](/integrations
* [**Delete** an execution](#delete-execution)
* Workflow
- * [**Activate** a workflow](#activate-deactivate-delete-and-get-workflow)
+ * [**Publish** a workflow](#activate-deactivate-delete-and-get-workflow)
* [**Create** a workflow](#create-workflow)
* [**Deactivate** a workflow](#activate-deactivate-delete-and-get-workflow)
</file context>
Deploying n8n-docs with
|
| Latest commit: |
ab17969
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://680527f4.n8n-docs-d9c.pages.dev |
| Branch Preview URL: | https://ado-4273-write-docs-draft.n8n-docs-d9c.pages.dev |
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
…igger/common-issues.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
No description provided. |
Summary by cubic
Standardized docs to use “publish/unpublish” for the workflow lifecycle, replacing “activate/deactivate” across the site. Added a dedicated “Publishing and saving workflows” page and linked it in nav.
New Features
Refactors
update:workflow; addedpublish:workflowandunpublish:workflow; removed--allto avoid accidental publishing.Written for commit ab17969. Summary will update automatically on new commits.