Skip to content

Multiple workflows#416

Merged
sriramveeraghanta merged 1 commit intomasterfrom
multiple-workflows
Mar 18, 2026
Merged

Multiple workflows#416
sriramveeraghanta merged 1 commit intomasterfrom
multiple-workflows

Conversation

@danciaclara
Copy link
Collaborator

@danciaclara danciaclara commented Mar 18, 2026

Summary by CodeRabbit

Documentation

  • Updated product tier branding ("Enterprise Grid") across documentation
  • Reorganized workflow documentation with clearer step-by-step guidance for configuring and managing workflows
  • Improved terminology consistency for workflow concepts and operations
  • Enhanced enterprise-specific feature callouts for approvals and custom workflows

@vercel
Copy link

vercel bot commented Mar 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 18, 2026 1:53pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

📝 Walkthrough

Walkthrough

Three documentation files updated: two files receive minor badge label changes from "Enterprise" to "Enterprise Grid," while the workflows documentation undergoes substantial rewording and restructuring, shifting terminology from "work item state transitions" to state management language with reorganized sections for workflow configuration.

Changes

Cohort / File(s) Summary
Badge Label Updates
docs/authentication/group-sync.md, docs/core-concepts/projects/manage-project-members.md
Updated badge labels from "Enterprise" to "Enterprise Grid" across two documentation files.
Workflows Documentation Rewrite
docs/workflows-and-approvals/workflows.md
Substantial content restructuring with terminology shifts from "work item transitions" to "states," new sections for workflow activation and management, expanded enterprise-specific messaging with Enterprise Grid badges, and reorganized guidance on workflow configuration and lifecycle.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 The docs hop along with renewed grace,
Badge labels dance to their rightful place,
Workflows now speak in clearer tongue,
States and transitions, newly sung!
Grid Enterprise shines bright and wide,
With rabbit-approved documentation pride! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Multiple workflows' is vague and generic, failing to convey the specific changes made across three documentation files. Provide a more descriptive title that captures the main changes, such as 'Update documentation for workflows configuration and enterprise features' or 'Refactor workflows documentation with new terminology and enterprise grid updates.'
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch multiple-workflows
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)
docs/core-concepts/projects/manage-project-members.md (1)

175-183: ⚠️ Potential issue | 🔴 Critical

Remove duplicate "How users join projects" section.

This section is identical to lines 138-147. The duplication creates confusion and makes the documentation harder to maintain. Remove one of the instances.

📝 Suggested fix

Delete lines 175-183 entirely, as they duplicate the section that already appears at lines 138-147.

-## How users join projects
-
-Users can become project members in two different ways, and understanding both helps you manage your project team effectively.
-
-**Direct project membership** is where you specifically invite users to your project and assign them roles. These members have access only to the projects you've added them to, and you have full control over their permissions.
-
-**Teamspace-based membership** happens automatically when your project is linked to a [Teamspace](/core-concepts/workspaces/teamspaces). All members of that teamspace automatically receive `Member` access to your project, making it perfect for teams that collaborate across multiple projects.
-
-Users can have both types of access simultaneously. When this happens, Plane automatically applies whichever role gives them higher permissions. For example, if someone is a `Guest` on your project but joins a linked teamspace, they're automatically upgraded to `Member` access. If they're already an `Admin`, they keep their `Admin` role.
-
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/core-concepts/projects/manage-project-members.md` around lines 175 -
183, Remove the duplicate "## How users join projects" block that repeats the
explanation of Direct project membership and Teamspace-based membership (the
header and the following paragraph describing Direct project membership,
Teamspace-based membership, and the upgrade/role-resolution example). Locate the
second occurrence of the "## How users join projects" header and delete that
entire duplicate section so only the original description remains.
docs/workflows-and-approvals/workflows.md (1)

18-31: ⚠️ Potential issue | 🟡 Minor

Remove or uncomment the embedded demo.

The commented-out iframe code suggests an Arcade demo was planned but not finalized. Either uncomment it if the demo is ready, or remove it entirely to keep the documentation clean.

🧹 Suggested fix

If the demo isn't ready, remove the commented block:

-<!--
-<div style="position: relative; padding-bottom: calc(56.67989417989418% + 41px); height: 0; width: 100%">
-  <iframe
-    src="https://demo.arcade.software/F2dAIS5BLVXcbgWUCQgl?embed&embed_mobile=tab&embed_desktop=inline&show_copy_link=true"
-    frameborder="0"
-    loading="lazy"
-    webkitallowfullscreen
-    mozallowfullscreen
-    allowfullscreen
-    style="position: absolute; top: 0; left: 0; width: 100%; height: 100%; color-scheme: light"
-    title="Set up your workspace"
-  ></iframe>
-</div>
--->
-
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/workflows-and-approvals/workflows.md` around lines 18 - 31, The
commented-out Arcade iframe demo block (the HTML <!-- ... <iframe
src="https://demo.arcade.software/..."> ...</iframe> ... -->) should be either
removed or uncommented: if the demo is not ready, delete the entire commented
block to keep docs clean; if the demo is ready, remove the HTML comment markers
so the iframe tag and its wrapper div are active and verify the iframe
attributes (src, title, loading, style) are correct and safe for embedding in
workflows.md.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@docs/core-concepts/projects/manage-project-members.md`:
- Around line 175-183: Remove the duplicate "## How users join projects" block
that repeats the explanation of Direct project membership and Teamspace-based
membership (the header and the following paragraph describing Direct project
membership, Teamspace-based membership, and the upgrade/role-resolution
example). Locate the second occurrence of the "## How users join projects"
header and delete that entire duplicate section so only the original description
remains.

In `@docs/workflows-and-approvals/workflows.md`:
- Around line 18-31: The commented-out Arcade iframe demo block (the HTML <!--
... <iframe src="https://demo.arcade.software/..."> ...</iframe> ... -->) should
be either removed or uncommented: if the demo is not ready, delete the entire
commented block to keep docs clean; if the demo is ready, remove the HTML
comment markers so the iframe tag and its wrapper div are active and verify the
iframe attributes (src, title, loading, style) are correct and safe for
embedding in workflows.md.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ba0cfc2b-a62a-4578-b8d0-d9f9602f19e5

📥 Commits

Reviewing files that changed from the base of the PR and between 4e8b5b2 and ff6e392.

📒 Files selected for processing (3)
  • docs/authentication/group-sync.md
  • docs/core-concepts/projects/manage-project-members.md
  • docs/workflows-and-approvals/workflows.md

@sriramveeraghanta sriramveeraghanta merged commit 25dea3b into master Mar 18, 2026
5 checks passed
@sriramveeraghanta sriramveeraghanta deleted the multiple-workflows branch March 18, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants