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.

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

adopt modern & supported Apollo Client patterns in the with-apollo example #65316

Merged
merged 4 commits into from
May 27, 2024

Conversation

phryneas
Copy link
Contributor

@phryneas phryneas commented May 3, 2024

#64186 removed the pages router implementation from the with-apollo example and replaced it with an app router implementation.

Unfortunately, it did so in an unsupported way.
At this point, we can not support any streaming SSR scenario without additional helper packages - in the case of the app router, it is vital to use the @apollo/experimental-nextjs-app-support package.

This PR switches the example to our officially supported patterns.

@phryneas phryneas requested review from a team as code owners May 3, 2024 08:27
@phryneas phryneas requested review from manovotny and delbaoliveira and removed request for a team May 3, 2024 08:27
@ijjk ijjk added the examples Issue was opened via the examples template. label May 3, 2024
@ijjk
Copy link
Member

ijjk commented May 3, 2024

Allow CI Workflow Run

  • approve CI run for commit: 98745e4

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@eps1lon eps1lon added the CI approved Approve running CI for fork label May 3, 2024
@phryneas
Copy link
Contributor Author

I don't want to stress anyone, but I really don't feel comfortable with the example that's currently out there.
Could someone please take a look?

@samcx samcx merged commit 73a05e1 into vercel:canary May 27, 2024
34 checks passed
@samcx samcx removed the CI approved Approve running CI for fork label May 27, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants