Conversation
according to https://specs.apollo.dev/link/v1.0/, we actually shouldn't be importing `@connect` — it's available by default when importing the spec. it can be renamed with `as:`. calling `import: ["@connect"]` should actually be importing something called `@connect__connect`, but we'll leave the current behavior. this change applies only to join__directive serialization.
|
|
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
dylan-apollo
left a comment
There was a problem hiding this comment.
Pretty sure I've done this before and it worked 🤔. Did something change at one point? Also, do we also need to update LSP?
|
it's never worked afaik |
|
i don't think there's anything to change in the language server? maybe a snippet but i don't see one for the link |
according to https://specs.apollo.dev/link/v1.0/, we actually shouldn't be importing
@connect— it's available by default when importing the spec. it can be renamed withas:. callingimport: ["@connect"]should actually be importing something called@connect__connect, but we'll leave the current behavior.this change applies only to join__directive serialization.