🐛 [Bug] WebpackError: ReferenceError: window is not defined

tuliooassis commented 4 years ago

User Story (if applicable)

As a I want to So that

Description

Describe the bug with a clear and concise description of what it is:

Error;
Impact;
Consequence;
How to fix;
Effort to fix.

Assumptions

Optionally, add assumptions here.

Steps to reproduce the behavior:

Go/navigate to '...';
Click/inspect on '....';
Scroll down to '....';
See error.

Expected behavior

(A clear and concise description of what you expected to happen)

Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Mobile (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Acceptance criteria

[ ] 1. User should...
Given
When
Then

Test scenarios

Add test scenarios.

E.g. try to reproduce this bug in devices like X and browsers like Y, feature toggle on/off...

Additional context

(Add any other context about the problem here, like screenshots)

Checklist do kickoff

[ ] A análise da história está concluída?
[ ] QA (ou pessoa que assumiu o papel de QA) revisou a análise?
[ ] A história está completa com todos os detalhes e as informações relevantes?
(Ex.: Contexto, Escopo, Fora de Escopo, Critérios de aceite, etc);
[ ] O valor de negócio da história está bem entendido? (se aplicável)
[ ] Esta história não tem dependência de outras histórias futuras?
[ ] Não existem dívidas técnicas (débitos técnicos) relacionadas a história;
[ ] Há protótipos de telas para essa história (se aplicável)
[ ] Há detalhes na história de mensagens de erro e outros feedbacks para o usuário (se aplicável)
[ ] Há mensagens de ajuda ou outros textos definidos na história (se aplicável)
[ ] A história está em um tamanho adequado
[ ] Existe feature toggle para a feature (se aplicável)

Checklist do deskcheck

[ ] Há testes unitários que validem a implementação? (se aplicável)
[ ] As alterações de código passaram em todas as pipelines
[ ] A história foi executada manualmente
[ ] A história não trouxe regressões ou impacto negativo a outras implementações
[ ] Todos os critérios de aceite foram cobertos
[ ] A história não precisa de feedback ou correção
[ ] Foi testada uma jornada completa do usuário através da história (se aplicável)

jimmyandrade commented 4 years ago

@tuliooassis Veja isso: https://github.com/gatsbyjs/gatsby/issues/309. Talvez possa ajudar.

tuliooassis commented 4 years ago

@jimmyandrade Verifiquei e vi que a causa foi a atualização da recoil neste merge.

Encontrei também uma issue no repo da lib.

Acredito que voltar para a versão 0.0.7 resolverá. O que acha?

jimmyandrade commented 4 years ago

@tuliooassis acho excelente! Parabéns e obrigado por ter achado esse problema! <3

Acabei de subir o commit https://github.com/multei/web/commit/a2c69d27fb5471d8dee03e1a99139e435ea05d4d, que volta para a versão 0.0.7 até o pessoal resolver isso. Nesse commit, também pedi ao Renovate para passar a ignorar updates do Recoil, para não corrermos o risco de ter esse problema de novo.

Já até fizeram uma PR em https://github.com/facebookexperimental/Recoil/pull/220 (e uma duplicata em https://github.com/facebookexperimental/Recoil/pull/233) para resolver isso. Vou acompanhar de perto e, quando lançarem uma correção (provavelmente na 0.0.9), a gente atualiza manualmente.

multei / web