fix #2485 Defined type from a basic type should not need scalar (#2486)

* fix #2485 remote named basics should not need scalar

* following review

* better way to compare basic type

_examples/scalars/external/model.go

@@ -1,3 +1,12 @@
 package external
 
-type ObjectID int
+type (
+	ObjectID     int
+	Manufacturer string // remote named string
+)
+
+const (
+	ManufacturerTesla  Manufacturer = "TESLA"
+	ManufacturerHonda  Manufacturer = "HONDA"
+	ManufacturerToyota Manufacturer = "TOYOTA"
+)

_examples/scalars/model/model.go

@@ -12,7 +12,11 @@ import (
 	"github.com/99designs/gqlgen/graphql"
 )
 
-type Banned bool
+type (
+	Banned      bool
+	LoginBanned bool
+	QueryBanned = bool
+)
 
 func (b Banned) MarshalGQL(w io.Writer) {
 	if b {
@@ -36,15 +40,18 @@ func (b *Banned) UnmarshalGQL(v interface{}) error {
 }
 
 type User struct {
-	ID       external.ObjectID
-	Name     string
-	Created  time.Time  // direct binding to builtin types with external Marshal/Unmarshal methods
-	Modified *time.Time // direct binding to builtin types with external Marshal/Unmarshal methods
-	ValPrefs Prefs      // external un/marshal that act on pointers
-	PtrPrefs *Prefs
-	IsBanned Banned
-	Address  Address
-	Tier     Tier
+	ID              external.ObjectID
+	Name            string
+	Created         time.Time  // direct binding to builtin types with external Marshal/Unmarshal methods
+	Modified        *time.Time // direct binding to builtin types with external Marshal/Unmarshal methods
+	ValPrefs        Prefs      // external un/marshal that act on pointers
+	PtrPrefs        *Prefs
+	IsBanned        Banned
+	IsLoginBanned   LoginBanned
+	IsQueryBanned   QueryBanned
+	Address         Address
+	Tier            Tier
+	CarManufacturer external.Manufacturer
 }
 
 // Point is serialized as a simple array, eg [1, 2]