-
Notifications
You must be signed in to change notification settings - Fork 19
docs: WebSocket subscriptions for AppSync #163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Added documentation for WebSocket subscriptions in AWS AppSync using LocalStack, including setup instructions and message types.
Deploying localstack-docs with
|
| Latest commit: |
3dcbddb
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4588a77d.localstack-docs.pages.dev |
| Branch Preview URL: | https://docs-websocket-subscriptions.localstack-docs.pages.dev |
cloutierMat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Lots of great work there! Thank you.
As we discussed, I am still unsure of the overall structure.
Possible change:
- GraphQL API
- Getting Started
- GraphQL resolvers
- Subscriptions (this could have the intro paragraph and the
supported message types - Subscription Example (not too sure of the name
- Configuring GraphQL endpoints
- GraphQL Resource Browser
It is not ideal either 🫤 WDYT?
Co-authored-by: Mathieu Cloutier <79954947+cloutierMat@users.noreply.github.com>
Ok, so having That said, I've made some adjustments in line with some of your suggestion above! Do you think the WebSocket Subscriptions should be h2, same level as GraphQL and Events API? Or do you think it should be h3?
Preview URL: https://4d703999.localstack-docs.pages.dev/aws/services/appsync/ cc @cloutierMat |
cloutierMat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @quetzalliwrites for addressing the previous comments! I think we are getting close, but I do have more suggestions that could improve the flow of the docs 😄
I personally think it should be H3, as it is a subset of GraphQl Apis. I don't think we should put that much emphasis on GraphQl WebSockets.
I also think that we shouldn't have both Configuring WebSocket Subscription endpoints and Configuring GraphQL endpoints as the WebSocket endpoint is just part of GraphQl APIs.
- GraphQL API
- Getting Started
- GraphQL resolvers
- WebSocket Subscriptions
- Supported WebSocket Message Type
- Configuring GraphQL endpoints (including WS endpoint)
- GraphQL Resource Browser
I think this hierarchy gives the user quick links to get where they need to. Having 2 h3 for WebSockets inside of GraphQl API seems plenty to me. And both endpoint style should fall in the same category, since they both answer the same question from a user.
Let me know if you have a good argument for keeping WebSocket Subscriptions as H2. But If we do, we should bring Configuring GraphQL endpoints and GraphQL Resource Browser in GraphQL API, which might make the flow a bit more bouncy?
Co-authored-by: Mathieu Cloutier <79954947+cloutierMat@users.noreply.github.com>
cloutierMat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Awesome! Thanks for the good back and forth! This looks great!

Added documentation for WebSocket subscriptions in AWS AppSync using LocalStack, including setup instructions and message types.
Preview URL:
https://9150cac2.localstack-docs.pages.dev/aws/services/appsync/#websocket-subscriptions