-
Notifications
You must be signed in to change notification settings - Fork 3.1k
feat(logs-context-menu): consolidated logs utils and types, added logs record context menu #2659
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.
|
…s record context menu
0104024 to
3a6dcc0
Compare
Contributor
Greptile SummaryConsolidated logs utilities and types into centralized locations and added a context menu for log rows with filtering, navigation, and copy actions. Key Changes:
Architecture Improvements: Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant LogsList
participant LogsPage
participant ContextMenu
participant FilterStore
participant Navigator
User->>LogsList: Right-click on log row
LogsList->>LogsPage: onContextMenu(event, log)
LogsPage->>LogsPage: setContextMenuPosition(x, y)
LogsPage->>LogsPage: setContextMenuLog(log)
LogsPage->>LogsPage: setContextMenuOpen(true)
LogsPage->>ContextMenu: Render with position and log data
alt Copy Execution ID
User->>ContextMenu: Click "Copy Execution ID"
ContextMenu->>LogsPage: onCopyExecutionId()
LogsPage->>LogsPage: navigator.clipboard.writeText(executionId)
ContextMenu->>ContextMenu: onClose()
end
alt Open Workflow
User->>ContextMenu: Click "Open Workflow"
ContextMenu->>LogsPage: onOpenWorkflow()
LogsPage->>Navigator: window.open(workflowUrl)
ContextMenu->>ContextMenu: onClose()
end
alt Filter by Workflow
User->>ContextMenu: Click "Filter by Workflow"
ContextMenu->>LogsPage: onToggleWorkflowFilter()
LogsPage->>FilterStore: setWorkflowIds([workflowId])
FilterStore-->>LogsPage: Filters updated
LogsPage-->>LogsList: Re-render with filtered logs
ContextMenu->>ContextMenu: onClose()
end
alt Clear All Filters
User->>ContextMenu: Click "Clear Filters"
ContextMenu->>LogsPage: onClearAllFilters()
LogsPage->>FilterStore: resetFilters()
LogsPage->>LogsPage: setSearchQuery('')
FilterStore-->>LogsPage: All filters cleared
LogsPage-->>LogsList: Re-render with unfiltered logs
ContextMenu->>ContextMenu: onClose()
end
|
waleedlatif1
added a commit
that referenced
this pull request
Jan 3, 2026
…ext menu (#2672) * feat(logs-context-menu): consolidated logs utils and types, added logs record context menu (#2659) * feat(email): welcome email; improvement(emails): ui/ux (#2658) * feat(email): welcome email; improvement(emails): ui/ux * improvement(emails): links, accounts, preview * refactor(emails): file structure and wrapper components * added envvar for personal emails sent, added isHosted gate * fixed failing tests, added env mock * fix: removed comment --------- Co-authored-by: waleed <walif6@gmail.com> * fix(logging): hitl + trigger dev crash protection (#2664) * hitl gaps * deal with trigger worker crashes * cleanup import strcuture * feat(imap): added support for imap trigger (#2663) * feat(tools): added support for imap trigger * feat(imap): added parity, tested * ack PR comments * final cleanup * feat(i18n): update translations (#2665) Co-authored-by: waleedlatif1 <waleedlatif1@users.noreply.github.com> * fix(grain): updated grain trigger to auto-establish trigger (#2666) Co-authored-by: aadamgough <adam@sim.ai> * feat(admin): routes to manage deployments (#2667) * feat(admin): routes to manage deployments * fix naming fo deployed by * feat(time-picker): added timepicker emcn component, added to playground, added searchable prop for dropdown, added more timezones for schedule, updated license and notice date (#2668) * feat(time-picker): added timepicker emcn component, added to playground, added searchable prop for dropdown, added more timezones for schedule, updated license and notice date * removed unused params, cleaned up redundant utils * improvement(invite): aligned styling (#2669) * improvement(invite): aligned with rest of app * fix(invite): error handling * fix: addressed comments --------- Co-authored-by: Emir Karabeg <78010029+emir-karabeg@users.noreply.github.com> Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com> Co-authored-by: waleedlatif1 <waleedlatif1@users.noreply.github.com> Co-authored-by: Adam Gough <77861281+aadamgough@users.noreply.github.com> Co-authored-by: aadamgough <adam@sim.ai>
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