Skip to content
This repository was archived by the owner on Apr 14, 2023. It is now read-only.

Fix: book-trips.tsx error#1142

Open
Barrosg99 wants to merge 1 commit intoapollographql:mainfrom
Barrosg99:patch-1
Open

Fix: book-trips.tsx error#1142
Barrosg99 wants to merge 1 commit intoapollographql:mainfrom
Barrosg99:patch-1

Conversation

@Barrosg99
Copy link

Missing '!' causes "Variable $launchIds of type [ID]! used in position expecting type [ID!]." error on book-trips.tsx

…sition expecting type [ID!]." error on book-trips.tsx
@apollo-cla
Copy link

@Barrosg99: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

@StephenBarlow
Copy link
Contributor

Hi @Barrosg99, I believe you might have put a ! in the incorrect spot in your bookTrips schema definition? This is the definition as written in the "Build a schema" chapter:

type Mutation {
  bookTrips(launchIds: [ID]!): TripUpdateResponse!
  cancelTrip(launchId: ID!): TripUpdateResponse!
  login(email: String): User
}

In terms of what the tutorial is expecting by default, I believe this is correct as-is 🤔

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants