-
Notifications
You must be signed in to change notification settings - Fork 3.2k
improvement(logs): improved logs ui bugs, added subflow disable UI #2910
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Contributor
Greptile SummaryThis PR adds the ability to enable/disable subflow blocks (loops and parallels) and improves the logs UI with better styling and centralized configuration. Key Changes
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant ActionBar
participant SubflowNode
participant WorkflowStore
participant PersistenceUtils
participant ExecutionEngine
Note over User,ExecutionEngine: Subflow Enable/Disable Feature
User->>ActionBar: Click enable/disable button
ActionBar->>WorkflowStore: collaborativeBatchToggleBlockEnabled([blockId])
WorkflowStore->>WorkflowStore: Update block.enabled field
WorkflowStore->>SubflowNode: Trigger re-render
SubflowNode->>SubflowNode: Update UI (gray icon, muted text, badge)
Note over User,ExecutionEngine: Logs UI Improvements
User->>User: View logs toolbar
Note over User: Status options now pulled from STATUS_CONFIG
User->>User: View log details
Note over User: Version badge shows light/dark mode styling
Note over User,ExecutionEngine: Workflow Save & Load
User->>WorkflowStore: Save workflow
WorkflowStore->>PersistenceUtils: Convert blocks to DB format
PersistenceUtils->>PersistenceUtils: Include enabled field in Loop/Parallel
PersistenceUtils->>ExecutionEngine: Store workflow with enabled state
ExecutionEngine->>PersistenceUtils: Load workflow
PersistenceUtils->>PersistenceUtils: Migrate enabled field (default: true)
PersistenceUtils->>WorkflowStore: Restore blocks with enabled state
WorkflowStore->>SubflowNode: Render with correct enabled state
|
waleedlatif1
added a commit
that referenced
this pull request
Jan 21, 2026
* fix(google): wrap primitive tool responses for Gemini API compatibility (#2900) * fix(canonical): copilot path + update parent (#2901) * fix(rss): add top-level title, link, pubDate fields to RSS trigger output (#2902) * fix(rss): add top-level title, link, pubDate fields to RSS trigger output * fix(imap): add top-level fields to IMAP trigger output * improvement(browseruse): add profile id param (#2903) * improvement(browseruse): add profile id param * make request a stub since we have directExec * improvement(executor): upgraded abort controller to handle aborts for loops and parallels (#2880) * improvement(executor): upgraded abort controller to handle aborts for loops and parallels * comments * improvement(files): update execution for passing base64 strings (#2906) * progress * improvement(execution): update execution for passing base64 strings * fix types * cleanup comments * path security vuln * reject promise correctly * fix redirect case * remove proxy routes * fix tests * use ipaddr * feat(tools): added textract, added v2 for mistral, updated tag dropdown (#2904) * feat(tools): added textract * cleanup * ack pr comments * reorder * removed upload for textract async version * fix additional fields dropdown in editor, update parser to leave validation to be done on the server * added mistral v2, files v2, and finalized textract * updated the rest of the old file patterns, updated mistral outputs for v2 * updated tag dropdown to parse non-operation fields as well * updated extension finder * cleanup * added description for inputs to workflow * use helper for internal route check * fix tag dropdown merge conflict change * remove duplicate code --------- Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai> * fix(ui): change add inputs button to match output selector (#2907) * fix(canvas): removed invite to workspace from canvas popover (#2908) * fix(canvas): removed invite to workspace * removed unused props * fix(copilot): legacy tool display names (#2911) * fix(a2a): canonical merge (#2912) * fix canonical merge * fix empty array case * fix(change-detection): copilot diffs have extra field (#2913) * improvement(logs): improved logs ui bugs, added subflow disable UI (#2910) * improvement(logs): improved logs ui bugs, added subflow disable UI * added duplicate to action bar for subflows * feat(broadcast): email v0.5 (#2905) --------- Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com> Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai> Co-authored-by: Emir Karabeg <78010029+emir-karabeg@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Type of Change
Testing
Tested manually
Checklist