cjoudrey/graphql-schema-linter

Non-null Int types for "first" argument are not accepted by rule "relay-connection-arguments-spec"

Open

#148 aperta il 7 set 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (62 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (669 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The following queries all fail lint checks:

  notablePeople(first: Int!, after: ID): NotablePersonConnection!

  users(first: Int!, after: ID, where: UsersQueryWhereInput): UserConnection!
    @requireOneOfRoles(roles: [MODERATOR])

  notablePeopleEvents(
    first: Int!
    after: ID
    where: NotablePersonEventsQueryWhereInput
  ): NotablePersonEventConnection!

If I remove the !, they all pass. Should not non-null integer types be accepted?

Guida contributor

Direzione di ricerca
Trova la regola di validazione 'relay connection arguments spec' e aggiorna la sua logica per accettare tipi Int non nulli per l'argomento 'first'.
Tech stack
javascriptgraphql
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GraphQLJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70