Closed EdimarioJr closed 5 months ago
Name | Link |
---|---|
Latest commit | d480d63cf87f4d6e04d58f9e034239bd7a162eed |
Latest deploy log | https://app.netlify.com/sites/querido-diario-plataforma/deploys/66048fb466611d0008d6c0ab |
Deploy Preview | https://deploy-preview-256--querido-diario-plataforma.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Ah, que alegria essa PR! ❤️ Obrigada pela contribuição @EdimarioJr
Atualizei a API do Querido Diário com novas cidades, sem atualizar o site, para ver o efeito aqui na PR e funcionou perfeitamente!
@lguima ou @kaiocp poderiam dar uma olhada se tudo ok nessa PR?
Obrigada pela PR @EdimarioJr e pela revisão @kaiocp! ❤️
Como entraram uns commits na main esses dias, só atualizei a branch.
Português (BR) | English (US)
Como não temos um endpoint de count de cidades, basicamente o que fiz foi usar o endpoint de listagem de cidades. Eu consumo esse endpoint na home e na página de cidades-disponiveis, porém para evitar chamadas desnecessárias ao back, adicionei um cache usando shareReply, então se o usuário entrar na home e depois entrar em /cidades-disponiveis, só uma chamada será disparada, e vice-versa.
Também modifiquei o json do content no componente de home para 'mergear' esse dado dinâmico aos dados estáticos do json sem precisar modificar o HTML.
Comunidade
Tipo de alteração
Issues relacionadas
Issues que são relacionadas a esta Pull Request.
194
Validação
Documentação