Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(server + client): add support for server-sent events (SSE) #5713

Open
wants to merge 334 commits into
base: next
Choose a base branch
from
Open

feat(server + client): add support for server-sent events (SSE) #5713

wants to merge 334 commits into from
+5,663 −1,143

Conversation

KATT
Copy link
Member

@KATT KATT commented May 10, 2024 •
edited

🎯 Changes

Adds a httpSubscriptionLink using server-sent events (SSE)

Docs: https://www-git-05-10-subscriptions-sse-trpc.vercel.app/docs/client/links/httpSubscriptionLink

✅ Checklist

  • I have followed the steps listed in the Contributing guide.
  • If necessary, I have added documentation related to the changes made.
  • I have added or updated the tests related to the changes made.

juliusmarminge
juliusmarminge reviewed Jun 12, 2024
View reviewed changes
www/docs/community/awesome-trpc.mdx
| **Recommended:** Starter project with Prisma, Next.js, tRPC, E2E-testing | https://github.com/trpc/examples-next-prisma-starter |
| **create-t3-turbo** - Clean and simple starter repo using the T3 Stack along with Expo React Native | http://github.com/t3-oss/create-t3-turbo |
| **create-t3-app** - Scaffold a starter project using the T3 Stack (Next.js, tRPC, Tailwind CSS, Prisma) | https://create.t3.gg |
| WebSockets Starter Project | https://github.com/trpc/examples-next-prisma-starter-websockets |
Copy link
Member

Choose a reason for hiding this comment

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

i'd keep this reference

juliusmarminge
juliusmarminge approved these changes Jun 12, 2024
View reviewed changes
Copy link
Member

@juliusmarminge juliusmarminge left a comment

Choose a reason for hiding this comment

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

lots of unfamiliar stuff so my review should be taken lightly but couldn't find anything obvious that stands out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sheraff Sheraff Sheraff left review comments

@BeBoRE BeBoRE BeBoRE left review comments

@juliusmarminge juliusmarminge juliusmarminge approved these changes

@helmturner helmturner Awaiting requested review from helmturner

Assignees
No one assigned
Labels
♻️ autoupdate
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consider SSE to enable server->client http subscriptions
6 participants
@KATT @helmturner @Adrastopoulos @Sheraff @BeBoRE @juliusmarminge