Closed alvarogfn closed 1 year ago
Name | Link |
---|---|
Latest commit | 31f1eb7b927fc7f54fde93f6e64fa1d129355e8a |
Latest deploy log | https://app.netlify.com/sites/dadosabertosdefeira/deploys/640b40a5db10e50008f1096a |
Deploy Preview | https://deploy-preview-128--dadosabertosdefeira.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 settings.
@alvarogfn eu acho que está ok, eu não lembro se fui eu que fiz a configuração por conta de nós gerarmos o build da página estática. Tenho que olhar, mas no geral parece ok. Posso olhar depois o porquê.
O que o Pull Request faz?
Atualizei o next para última versão stable (13.2.3) (segui o manual de migração).
Removi o componente customizado de Link. Foi corrigido pelo Next13.
Removi o Next Compose Plugin, pois a biblioteca não é atualizada a mais de 2 anos e não tem suporte para a versão 13 do nextjs. (Configurei o next.config.js com base nesse artigo).
Atualizei a React Testing Library para a versão 14, pois o suporte para React <= 17 foi largado na versão 13.
Removi o Babelrc para utilizar o novo compilador SWC. Apesar disso, o Jest não funciona sem uma configuração de babel, então adicionei diretamente no jest.config.js.
Corrigi uma formatação do eslint (Só para poder commitar).
Atualizei as bibliotecas do netlify do next.
Por quê o Pull Request foi aberto?
Manter as dependências do projeto atualizadas.
Notas Adicionais
Eu não entendi muito bem a configuração do Sentry, e estava dando uma olhada na documentação, e notei algumas coisas.
E a configuração de target = "serveless" foi removida do nextjs nesse pull. O projeto estava usando essa configuração. Podem dar uma olhada no next.config.js para ver se está tudo funcionando bem?