Skip to content

Light editorial around delivery agnostic subscriptions#959

Merged
leebyron merged 2 commits intomainfrom
subscription-algorithms
Jun 16, 2022
Merged

Light editorial around delivery agnostic subscriptions#959
leebyron merged 2 commits intomainfrom
subscription-algorithms

Conversation

@benjie
Copy link
Copy Markdown
Member

@benjie benjie commented Jun 5, 2022

Previously:

Subscriptions specifies algorithms for the creation of a stream, the content of each payload on that stream, and the closing of that stream.

The phrase "Subscriptions specifies" here was confusing, it's the GraphQL spec that does the specification, so I have clarified:

GraphQL specifies algorithms for the creation of the response stream, the content of each payload on that stream, and the closing of that stream.

This is an alternative to a change in #957

@benjie benjie added the ✏️ Editorial PR is non-normative or does not influence implementation label Jun 5, 2022
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 5, 2022

Deploy Preview for graphql-spec-draft ready!

Name Link
🔨 Latest commit ed6ce6c
🔍 Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/62a4befcd998c80008b41d96
😎 Deploy Preview https://deploy-preview-959--graphql-spec-draft.netlify.app/draft
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@leebyron leebyron merged commit a91e158 into main Jun 16, 2022
@leebyron leebyron deleted the subscription-algorithms branch June 16, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✏️ Editorial PR is non-normative or does not influence implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants