Skip to content

Conversation

@acao
Copy link
Member

@acao acao commented Nov 22, 2024

fixes #2405

switching to a type import will prevent bundling issues with legacy bundlers, as the older babel/ts transforms will know to strip type only imports before build time

@changeset-bot
Copy link

changeset-bot bot commented Nov 22, 2024

🦋 Changeset detected

Latest commit: b94ff26

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

This PR includes changesets to release 1 package
Name Type
@graphiql/toolkit 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

@github-actions
Copy link
Contributor

github-actions bot commented Nov 22, 2024

The latest changes of this PR are available as canary in npm (based on the declared changesets):

@acao acao force-pushed the fix-2405-graphql-ws-import branch from d6dbcc1 to b94ff26 Compare November 22, 2024 05:12
@acao acao merged commit 5f2c3a4 into main Nov 22, 2024
14 checks passed
@acao acao deleted the fix-2405-graphql-ws-import branch November 22, 2024 05:17
@acao acao mentioned this pull request Nov 22, 2024
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.

Can't resolve graphql-ws from @graphiql/toolkit/esm/create-fetcher

2 participants