Merge pull request #136 from jeddeloh/missing-name-on-operationdefini…

…tionnode Account for missing name on OperationDefinitionNode
jeddeloh · Dec 23, 2021 · a2697fa · a2697fa
2 parents 2c17c20 + 49b41ad
commit a2697fa
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/EXAMPLES/package.json b/EXAMPLES/package.json
@@ -10,7 +10,7 @@
     "start": "bsb -make-world -w"
   },
   "devDependencies": {
-    "@reasonml-community/graphql-ppx": "1.0.0",
+    "@reasonml-community/graphql-ppx": "1.2.2",
     "bs-platform": "9.0.0",
     "graphql-client-example-server": "1.2.0",
     "html-webpack-plugin": "4.3.0",
@@ -19,14 +19,14 @@
     "webpack-dev-server": "3.11.0"
   },
   "dependencies": {
-    "@apollo/client": "3.3.21",
+    "@apollo/client": "3.5.6",
     "@rescript/react": "0.10.1",
     "@ryyppy/rescript-promise": "0.0.2",
     "graphql": "15.3.0",
     "react": "16.13.1",
     "react-dom": "16.13.1",
     "reason-promise": "1.1.1",
-    "rescript-apollo-client": "2.4.0",
+    "rescript-apollo-client": "../",
     "subscriptions-transport-ws": "0.9.16"
   }
 }
diff --git a/EXAMPLES/src/hooksUsage/Query_SubscribeToMore.res b/EXAMPLES/src/hooksUsage/Query_SubscribeToMore.res
@@ -26,7 +26,7 @@ let make = () => {
    * Sorry, this example is nonsensical given the current schema, but I'm gonna proceed anyway
    ")
   React.useEffect0(() => {
-    queryResult.subscribeToMore(
+    let _unsubscribe = queryResult.subscribeToMore(
       ~subscription=module(SorryItsNotASubscriptionForTodos),
       ~updateQuery=(previous, {subscriptionData: {data: {siteStatisticsUpdated}}}) => {
         let count =

diff --git a/src/graphql/language/ApolloClient__Graphql_Language_Ast.res b/src/graphql/language/ApolloClient__Graphql_Language_Ast.res
@@ -172,7 +172,7 @@ module OperationDefinitionNode = {
   type t = {
     kind: string,
     loc: option<Location.t>,
-    name: NameNode.t,
+    name: option<NameNode.t>,
     operation: OperationTypeNode.t,
     variableDefinitions: option<array<VariableDefinitionNode.t>>,
     directives: option<array<DirectiveNode.t>>,