Skip to content

Manual refresh of call stack query#3330

Merged
Alex-Tideman merged 1 commit intomainfrom
manual-call-stack-refresh
Apr 22, 2026
Merged

Manual refresh of call stack query#3330
Alex-Tideman merged 1 commit intomainfrom
manual-call-stack-refresh

Conversation

@Alex-Tideman
Copy link
Copy Markdown
Collaborator

@Alex-Tideman Alex-Tideman commented Apr 21, 2026

Description & motivation 💭

Don't automatically refresh the call stack query every 10 seconds. Let the user trigger the refresh of the call stack so they can read the call stack without it changing on them.

Screenshots (if applicable) 📸

Screenshot 2026-04-21 at 12 50 47 PM

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

@Alex-Tideman Alex-Tideman requested a review from a team as a code owner April 21, 2026 17:55
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

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

Project Deployment Actions Updated (UTC)
holocene Ready Ready Preview, Comment Apr 21, 2026 5:55pm

Request Review

@Alex-Tideman Alex-Tideman changed the title Add manual refresh of call stack Manual refresh of call stack query Apr 21, 2026
let refreshDate = $state<string>();

const getStackTrace = () =>
getWorkflowStackTrace({
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • ⚠️ Type 'WorkflowExecution | null' is not assignable to type 'Eventual<{ id: string; runId: string; }>'.

@temporal-cicd
Copy link
Copy Markdown
Contributor

temporal-cicd Bot commented Apr 21, 2026

Warnings
⚠️

📊 Strict Mode: 3 errors in 1 file (0.3% of 997 total)

src/lib/pages/workflow-call-stack.svelte (3)
  • L26:6: Type 'WorkflowExecution | null' is not assignable to type 'Eventual<{ id: string; runId: string; }>'.
  • L45:30: 'workers.pollers.length' is possibly 'undefined'.
  • L108:12: Type 'string | undefined' is not assignable to type 'string | number'.

Generated by 🚫 dangerJS against e4301b7

@Alex-Tideman Alex-Tideman merged commit 0361e3e into main Apr 22, 2026
22 checks passed
@Alex-Tideman Alex-Tideman deleted the manual-call-stack-refresh branch April 22, 2026 20:08
rossnelson added a commit that referenced this pull request Apr 22, 2026
Auto-generated version bump from 2.49.0 to 2.49.1

Bump type: patch

Changes included:
- [`fccd87f9`](fccd87f) Bump Go toolchain to 1.26.2 for CVE fix (#3336)
- [`0361e3e9`](0361e3e) Add manual refresh of call stack (#3330)
- [`17293020`](1729302) refactor: move fetchDeployment into Deployment component (#3340)

Co-authored-by: rossnelson <146089+rossnelson@users.noreply.github.com>
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