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

…aphjin queries (#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]>
dosco · Dec 7, 2023 · 5d40427 · 5d40427 · vercel · Dec 7, 2023
1 parent 347b070
commit 5d40427
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/core/intro.go b/core/intro.go
@@ -293,10 +293,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
 }