Esse PR corrige o site do Finacial Times que estava "mais ou menos funcionando". Mais ou menos pq era preciso dar um refresh na página depois de ser bloqueado. Isso pq o script estava apagando os cookies e localStorage antes da página setar os valores iniciais.
Eu precisei criar uma função extra (eraseAllCookiesExcept) para apagar todos os cookies com a exceção de 2 que eram necessários para fazer sumir um maldito modal que perguntava sobre as opções de cookies sumir (não consegui sumir com esse modal de nenhuma outra forma...).
Como utilizei as funções antigas nessa função nova, acabei refatorando as outras duas funções eraseAllCookies e eraseCookie. Acho que está muito mais fácil de entender o que elas fazem agora. Se preferir, eu faço um PR separado para elas...
Não sei se o problema que eu estava tendo no site era o mesmo dessa issue #12, que é bem antiga, mas achei que dava pra fechá-la mesmo assim.
Uma última coisa: acabei desligando a regra de tamanho máximo da linha do eslint, uma vez que tinham vários warnings e pelo jeito, vcs não fazem questão que essa regra seja seguida. Então, achei melhor desligar para sumir com os warnings.
kemelzaidan
commented
4 months ago
Esse PR corrige o site do Finacial Times que estava "mais ou menos funcionando". Mais ou menos pq era preciso dar um refresh na página depois de ser bloqueado. Isso pq o script estava apagando os cookies e localStorage antes da página setar os valores iniciais.
Eu precisei criar uma função extra (
eraseAllCookiesExcept) para apagar todos os cookies com a exceção de 2 que eram necessários para fazer sumir um maldito modal que perguntava sobre as opções de cookies sumir (não consegui sumir com esse modal de nenhuma outra forma...).Como utilizei as funções antigas nessa função nova, acabei refatorando as outras duas funções
eraseAllCookieseeraseCookie. Acho que está muito mais fácil de entender o que elas fazem agora. Se preferir, eu faço um PR separado para elas...Não sei se o problema que eu estava tendo no site era o mesmo dessa issue #12, que é bem antiga, mas achei que dava pra fechá-la mesmo assim.
Uma última coisa: acabei desligando a regra de tamanho máximo da linha do eslint, uma vez que tinham vários warnings e pelo jeito, vcs não fazem questão que essa regra seja seguida. Então, achei melhor desligar para sumir com os warnings.