Skip to content

Conversation

yaacovCR
Copy link
Contributor

Currently, they are presented as errors in the next pending item.

Of course, there may or not be a next pending item; there may not be a way to definitively know the reason for the error, such that the error is a function of a problem with the collection itself.

As discussed in a prior incremental delivery WG meeting, this must be changed for both the non-streaming and streaming cases; this PR makes the minor adjustments addressing both cases.

…ection itself, not within the next pending item

as discussed in a prior incremental delivery WG meeting, this must be changed for both the non-streaming and streaming cases
@yaacovCR yaacovCR requested a review from IvanGoncharov May 23, 2023 10:13
@netlify
Copy link

netlify bot commented May 23, 2023

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit 2535ed4
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/646c91b718ee890008ea4962
😎 Deploy Preview https://deploy-preview-3899--compassionate-pike-271cb3.netlify.app
📱 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.

@yaacovCR yaacovCR requested review from robrichard and a team May 23, 2023 10:13
@yaacovCR yaacovCR added the PR: bug fix 🐞 requires increase of "patch" version number label May 23, 2023
@github-actions
Copy link

Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment:

  • @github-actions run-benchmark - Run benchmark comparing base and merge commits for this PR
  • @github-actions publish-pr-on-npm - Build package from this PR and publish it on NPM

@yaacovCR yaacovCR changed the title asyncIterable: async iterator errors are located within the collection asyncIterable: locate async iterator errors to the collection May 23, 2023
@yaacovCR yaacovCR merged commit bd558cb into graphql:main May 23, 2023
@yaacovCR yaacovCR deleted the error-collection branch May 23, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: bug fix 🐞 requires increase of "patch" version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants