cjoudrey/graphql-schema-linter

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

Ouverte

#148 ouverte le 7 sept. 2018

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)JavaScript (61 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (704 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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?

Guide contributeur

Direction de recherche
Trouvez la règle de validation 'relay connection arguments spec' et mettez à jour sa logique pour accepter les types Int non nuls pour l'argument 'first'.
Stack technique
javascriptgraphql
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GraphQLJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70