Skip to content

chore: upgrade telegraph packages to fix dashboard dep conflict#903

Merged
kylemcd merged 1 commit intomainfrom
kyle-kno-12049-update-javascript-sdk-to-latest-telegraph-versions
Mar 13, 2026
Merged

chore: upgrade telegraph packages to fix dashboard dep conflict#903
kylemcd merged 1 commit intomainfrom
kyle-kno-12049-update-javascript-sdk-to-latest-telegraph-versions

Conversation

@kylemcd
Copy link
Member

@kylemcd kylemcd commented Mar 13, 2026

Description

There's an issue in the dashboard with focus states in the combobox. This is stemming from there being 2 different versions of @telegraph/button in our dependency tree because one exists in this package. So this PR updates those telegraph deps so that it's on the new version with the updated pseudo element logic.

@linear
Copy link

linear bot commented Mar 13, 2026

@vercel
Copy link

vercel bot commented Mar 13, 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 13, 2026 3:14pm
javascript-nextjs-example Ready Ready Preview, Comment Mar 13, 2026 3:14pm
javascript-slack-connect-example Ready Ready Preview, Comment Mar 13, 2026 3:14pm
javascript-slack-kit-example Ready Ready Preview, Comment Mar 13, 2026 3:14pm

Request Review

Copy link
Member Author

kylemcd commented Mar 13, 2026

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

@changeset-bot
Copy link

changeset-bot bot commented Mar 13, 2026

🦋 Changeset detected

Latest commit: 27d4990

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

@meryldakin meryldakin left a comment

Choose a reason for hiding this comment

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

nice

@kylemcd kylemcd merged commit 4cc5bac into main Mar 13, 2026
14 checks passed
@kylemcd kylemcd deleted the kyle-kno-12049-update-javascript-sdk-to-latest-telegraph-versions branch March 13, 2026 15:18
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.15%. Comparing base (796e7f0) to head (27d4990).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #903   +/-   ##
=======================================
  Coverage   65.15%   65.15%           
=======================================
  Files         207      207           
  Lines        9593     9593           
  Branches     1250     1250           
=======================================
  Hits         6250     6250           
  Misses       3318     3318           
  Partials       25       25           

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.

3 participants