This repository has been archived by the owner on Aug 14, 2020. It is now read-only.
#19 initial API calls config #92
Merged
pabloklein
merged 15 commits into
CovidZero:master
from
LucasGarcez:feaute/#19-API-calls
Mar 27, 2020
Merged
#19 initial API calls config #92
pabloklein
merged 15 commits into
CovidZero:master
from
LucasGarcez:feaute/#19-API-calls
Mar 27, 2020
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update 20/03
update fork 21/03
Merge CovidZero
jeduardo824
suggested changes
Mar 27, 2020
Resolve também os conflitos com a branch base, por favor. |
Resolvido. Corrigi os demais problemas tbm que foram apontados. |
augustolima1
approved these changes
Mar 27, 2020
augustolima1
approved these changes
Mar 27, 2020
augustolima1
approved these changes
Mar 27, 2020
jeduardo824
reviewed
Mar 27, 2020
function API() { | ||
|
||
const baseURLs = { | ||
cases: `https://api.covidzero.com.br/data_api/v1/data` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seria bom n deixar essa URL hardcoded, mas enfim, aceito.
jeduardo824
approved these changes
Mar 27, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Criei uma estrutura básica para concentrar as chamadas a API. Como ainda temos muita coisa para corrigir, achei q seria um pouco boilerplate subir essas chamadas já acoplada na estrutura do redux com sagas e tudo mais.
E já preparei as chamadas pra receber os dados tbm de suspeitos e recuperadas, então quando o backend incluir já vai aparecer lá sem precisar fazer qualquer modificação.
Adicionei Typescript ao projeto para facilitar o uso dos dados ao longo do app sem que os demais devs precisem ficar consultado os endpoins a todo momento.
Também criei algumas funções simples apenas para converter o nome das props e deixá-las mais amigáveis e tratar dados que possívelmente esperamos (como no casos de suspeitos e recuperados), mas q ainda não estão aparecendo no data do response:
Então criei duas interfaces, uma que representa exatamente os dados vindos do Response:
E outra que será de fato utilizada na aplicação com os dados da maneira que queremos apresenta-los:
Na tela do Map utilizei o endpoint dos casos por estados ao invés do geral, por creio q em breve ele será necessário por lá.
Temos muita coisa pra mudar ainda, mas creio que essa seja uma boa estrutura para um ponto de partida e destravarmos essa parte.
OBS: Ainda enfrentamos o problema com o CORS para acessar a API, apesar da requisição me retornar 200 no status code, o axios ainda está lançando um network error. Para driblar esse problema usei a sugestão do amigo lá no Slack e instale a extensão do chrome CORS Unblock.