Fix: issue where all primary key queries were mandatory in default gr…

…aphjin queries (dosco#468) * Updated the basic documentation example * Updated addTable function to create a new variable that would create the type that has not_null for id in the scenario where we query by ID * Update intro.go * Update intro.go * Update intro.go --------- Co-authored-by: Spacy <[email protected]>
rkrishnasanka · Apr 15, 2024 · 2e3443a · 2e3443a
1 parent 46326df
commit 2e3443a
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/core/intro.go b/core/intro.go
@@ -295,10 +295,19 @@ func (in *Introspection) addTable(table sdata.DBTable, alias string) (err error)
 	}
 	in.addTypeTo("Mutation", ftM)
 
-	ftQS.addOrReplaceArg("id", newTypeRef(KIND_NONNULL, "", newTypeRef("", "ID", nil)))
-	in.addType(ftQS)
-	in.addTypeTo("Query", ftQS)
-	in.addTypeTo("Subscription", ftQS)
+	// add tableByID type to query and subscription
+	var ftQSByID FullType
+
+	if ftQSByID, err = in.addTableType(table, alias); err != nil {
+		return
+	}
+
+	ftQSByID.Name += "ByID"
+	ftQSByID.addOrReplaceArg("id", newTypeRef(KIND_NONNULL, "", newTypeRef("", "ID", nil)))
+	in.addType(ftQSByID)
+	in.addTypeTo("Query", ftQSByID)
+	in.addTypeTo("Subscription", ftQSByID)
+
 	return
 }