Skip to content

feat: handle focused guide key param in guide toolbar run config#888

Open
thomaswhyyou wants to merge 5 commits intomainfrom
thomas-kno-11802-toolbar-support-focusing-on-a-target-guide-2
Open

feat: handle focused guide key param in guide toolbar run config#888
thomaswhyyou wants to merge 5 commits intomainfrom
thomas-kno-11802-toolbar-support-focusing-on-a-target-guide-2

Conversation

@thomaswhyyou
Copy link
Contributor

@thomaswhyyou thomaswhyyou commented Mar 4, 2026

Description

This PR handles the focused_guide_key param as part of the guide toolbar run config. This param is used in https://github.com/knocklabs/control/pull/7683 and https://github.com/knocklabs/control/pull/7687 when previewing an individual guide. Clicking on the Open preview button in dashboard opens a new tab with the configured preview url and sets the query param like focused_guide_key=foo. And the changes here allows the toolbar to boot with the focused guide key, allowing it to force render that guide when it opens.

Screenshots or videos

CleanShot.2026-03-04.at.11.13.57.mp4

@vercel
Copy link

vercel bot commented Mar 4, 2026

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

Project Deployment Actions Updated (UTC)
javascript-ms-teams-connect-example Ready Ready Preview, Comment Mar 4, 2026 8:02pm
javascript-nextjs-example Ready Ready Preview, Comment Mar 4, 2026 8:02pm
javascript-slack-connect-example Ready Ready Preview, Comment Mar 4, 2026 8:02pm
javascript-slack-kit-example Ready Ready Preview, Comment Mar 4, 2026 8:02pm

Request Review

@linear
Copy link

linear bot commented Mar 4, 2026

@changeset-bot
Copy link

changeset-bot bot commented Mar 4, 2026

🦋 Changeset detected

Latest commit: ab4dfd0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@knocklabs/react Patch
guide-example Patch
ms-teams-connect-example Patch
nextjs-app-dir-example Patch
nextjs-example Patch
slack-connect-example Patch
slack-kit-example Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@thomaswhyyou
Copy link
Contributor Author

@cursor review

@thomaswhyyou thomaswhyyou changed the title handle focused guide params in run config feat: handle focused guide params in run config Mar 4, 2026
@thomaswhyyou thomaswhyyou changed the title feat: handle focused guide params in run config feat: handle focused guide key param in guide toolbar run config Mar 4, 2026
@thomaswhyyou
Copy link
Contributor Author

@cursor review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

@thomaswhyyou thomaswhyyou marked this pull request as ready for review March 4, 2026 18:37
@thomaswhyyou thomaswhyyou requested review from a team, andy-knock and connorlindsey and removed request for a team March 4, 2026 18:37
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

❌ Patch coverage is 81.08108% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.06%. Comparing base (c134f53) to head (ab4dfd0).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...act/src/modules/guide/components/Toolbar/V2/V2.tsx 0.00% 6 Missing ⚠️
...src/modules/guide/components/Toolbar/V2/helpers.ts 93.75% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #888      +/-   ##
==========================================
+ Coverage   66.99%   67.06%   +0.06%     
==========================================
  Files         207      207              
  Lines        9224     9245      +21     
  Branches     1238     1247       +9     
==========================================
+ Hits         6180     6200      +20     
- Misses       3020     3021       +1     
  Partials       24       24              
Files with missing lines Coverage Δ
...omponents/Toolbar/V2/useInspectGuideClientStore.ts 100.00% <100.00%> (ø)
...src/modules/guide/components/Toolbar/V2/helpers.ts 94.33% <93.75%> (+0.72%) ⬆️
...act/src/modules/guide/components/Toolbar/V2/V2.tsx 14.17% <0.00%> (-0.12%) ⬇️

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