graphql-spec icon indicating copy to clipboard operation
graphql-spec copied to clipboard

Published 20 hours ago • verified publisher icon graphql

Fix normative requirement around `errors` property

Open benjie opened this issue 3 years ago • 1 comments

Whilst reviewing #966 I spotted this; I don't believe this is a breaking change since it can be derived from:

The errors entry in the response is a non-empty list of errors, where each error is a map.

So if no errors occur, then the "errors" key must not (rather than "should not") be present, since it's forbidden to be an empty list, it contains errors, and no errors occurred.

benjie avatar Jun 22 '22 07:06 benjie

Deploy Preview for graphql-spec-draft ready!

Name Link
Latest commit 7f541ddf1a58dd2fe915028c3ffa759426107ab2
Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/62b2c3763891f50009a2a9ac
Deploy Preview https://deploy-preview-967--graphql-spec-draft.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.

netlify[bot] avatar Jun 22 '22 07:06 netlify[bot]

Changes were incorporated into #966

benjie avatar Sep 01 '22 10:09 benjie