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

Store and combine incremental response data #3147

Closed
Tracked by #3093
calvincestari opened this issue Jul 24, 2023 · 2 comments · Fixed by apollographql/apollo-ios-dev#217
Closed
Tracked by #3093

Store and combine incremental response data #3147

calvincestari opened this issue Jul 24, 2023 · 2 comments · Fixed by apollographql/apollo-ios-dev#217
Assignees

Comments

@calvincestari
Copy link
Member

calvincestari commented Jul 24, 2023

All responses (initial and incremental) are parsed through the parsers and executor. All data received prior to each response will need to be combined with the new data received so a complete response is formed - RFC section link.

The initial response data and data received in each incremental response will need to be retained and combined so that each incremental response can insert the latest received incremental response data at the correct path and return an up-to-date response to the request callback.

@calvincestari
Copy link
Member Author

This was done in PR apollographql/apollo-ios-dev#217.

Copy link

Do you have any feedback for the maintainers? Please tell us by taking a one-minute survey. Your responses will help us understand Apollo iOS usage and allow us to serve you better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant