Merge pull request #1227 from whysage/patch_learn_serving_over_http

orta · web-flow · commit 77fe87e3917f · 2022-04-14T18:07:02.000+01:00
docs: fix data/errors block
diff --git a/src/content/learn/BestPractice-ServingOverHTTP.md b/src/content/learn/BestPractice-ServingOverHTTP.md
@@ -69,7 +69,7 @@ Regardless of the method by which the query and variables were sent, the respons
 }
 ```
 
-If there were no errors returned, the `"errors"` field should not be present on the response. If no data is returned, [according to the GraphQL spec](http://facebook.github.io/graphql/#sec-Data), the `"data"` field should only be included if the error occurred during execution.
+If there were no errors returned, the `"errors"` field should not be present on the response. If no data is returned, [according to the GraphQL spec](http://facebook.github.io/graphql/#sec-Data), the `"data"` field should only be included if no errors occurred during execution.
 
 ## GraphiQL
 GraphiQL is useful during testing and development but should be disabled in production by default. If you are using express-graphql, you can toggle it based on the NODE_ENV environment variable:

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ Regardless of the method by which the query and variables were sent, the respons`
`69`	`69`	`}`
`70`	`70`	```
`71`	`71`
`72`		-If there were no errors returned, the `"errors"` field should not be present on the response. If no data is returned, [according to the GraphQL spec](http://facebook.github.io/graphql/#sec-Data), the `"data"` field should only be included if the error occurred during execution.
	`72`	+If there were no errors returned, the `"errors"` field should not be present on the response. If no data is returned, [according to the GraphQL spec](http://facebook.github.io/graphql/#sec-Data), the `"data"` field should only be included if no errors occurred during execution.
`73`	`73`
`74`	`74`	`## GraphiQL`
`75`	`75`	`GraphiQL is useful during testing and development but should be disabled in production by default. If you are using express-graphql, you can toggle it based on the NODE_ENV environment variable:`