google/rejoiner

Schema Modification does not work on Input types

Open

#70 aberto em 27 de fev. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Java (152 forks)batch import
help wanted

Métricas do repositório

Stars
 (3.647 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hello,

We are trying to remove a field on an input by doing SchemaModification, but from this code it looks like proto modification only works on GraphQLObjectType instead of GraphQLInputObjectType:

https://github.com/google/rejoiner/blob/cedc123c18918d8c090543a4236ee344d2dbffeb/rejoiner/src/main/java/com/google/api/graphql/rejoiner/ProtoRegistry.java#L196-L206

Is there any chance we can add support for type modification on the input type?

Guia do colaborador

Direção de pesquisa
Examine a seção de código vinculada em ProtoRegistry.java e modifique a lógica de modificação de esquema para também lidar com GraphQLInputObjectType.
Pilha de tecnologia
java
Domain
apibackend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaGraphQLProtobuf
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60