Endpoint graphql #43
Endpoint graphql #43
Conversation
|
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/filipedeschamps/brasilapi/ir3ffcqlk |
Bumps [next](https://github.com/zeit/next.js) from 9.3.0 to 9.3.2. - [Release notes](https://github.com/zeit/next.js/releases) - [Commits](vercel/next.js@v9.3.0...v9.3.2) Signed-off-by: dependabot[bot] <support@github.com>
…nd_yarn/next-9.3.2 build(deps): bump next from 9.3.0 to 9.3.2
Bumps [acorn](https://github.com/acornjs/acorn) from 6.4.0 to 6.4.1. - [Release notes](https://github.com/acornjs/acorn/releases) - [Commits](acornjs/acorn@6.4.0...6.4.1) Signed-off-by: dependabot[bot] <support@github.com>
…nd_yarn/acorn-6.4.1 build(deps): bump acorn from 6.4.0 to 6.4.1
|
Sensacional!!! Estou pensando aqui, você acha muito ruim deixar as implementações de GraphQL e REST totalmente separadas/independentes? Pois imagino que a carga de esforço nas duas vai ser diferente, digo quanto a disponibilidade de contribuição. E amarrar a disponibilidade de uma mesma informação por conta de duas tecnologias diferentes acredito não ser saudável para o momento do projeto. O que você acha? |
|
Eu acho muito bom, só fico meio preocupado que se começarem a implementar tudo em REST sem atualizar o graphql ele corre o risco de ficar defasado. Porem podemos ter mantenedores que cuidem de atualizar o graphql. |
* 'master' of github.com:WeslleyNasRocha/BrasilAPI: fix: moved graphql to modules fix: cache-control header docs: documentation for the cep query feat: added graphql endpoint
Show, essa é uma preocupação minha também, porém acredito ser melhor esse descasamento (para revelar que precisamos de mais contribuidores para parte do GraphQL) sem penalizar ou deixar mais difícil a implementação do endpoint em REST 👍 Estou fazendo o merge, mesmo com o |
Acredito que vai ajudar bastante quem quer usar graphql com essa api