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

Fix: book-trips.tsx error #1142

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

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.

None yet

3 participants