search

ecomplus / app-offers-notification

E-Com Plus app for products notifications to subscribers
MIT License
0 stars 0 forks source link

Parou de funcionar avisa-me ou notificações #13

Closed matheusgnreis closed 3 years ago

matheusgnreis commented 3 years ago

avisa-me Exemplo https://storefront-demo.e-com.plus/combo-teclado-e-mouse-thermaltake-kb-cmc-plblpb-01-preto

leomp12 commented 3 years ago

@matheusgnreis nesse link aí está funcionando :thinking:

matheusgnreis commented 3 years ago

https://belabruna.com.br/vestido-argola-envelope-animal-print Em alguns funcionam em guia anônima apenas. Esse aí não consegui, nem o lojista

talissonf commented 3 years ago

@matheusgnreis eu testei aqui e funcionou em guia normal e anônima.

matheusgnreis commented 3 years ago

@talissonf fato é que não está funcionando pra ninguem fora vc e leo, mais uma abertura de ticket na comunidade sobre isso.

matheusgnreis commented 3 years ago

Já testei em safari no macbook, no windows via chrome e nada. Só quando coloco guia anonima

talissonf commented 3 years ago

@matheusgnreis quando ocorrer isso tenta ver o console do popup pra ver se tem alguma informação relevante porque o app não tem log de erros pra isso.

matheusgnreis commented 3 years ago

É o adblock, está indentificando como link de propaganda. No caso do iframe deve resolver, pelo teste q fiz

talissonf commented 3 years ago

@matheusgnreis @leomp12 removi o jquery como dependência e uns javascript do boostrap tmb. Ta tudo em vanila agora, acho q esse problema com adblock foi resolvido.

Tambem inclui um spiner no iframe pra indicar que está sendo carregado https://talisson.ngrok.io/

leomp12 commented 3 years ago

@talissonf até apareceu um spinner mas ele some bem antes de o formulário ficar visível, talvez deveria ter um spinner na view do app também até que esteja toda renderizada... Além disso o iframe ainda está muito pequeno eu acho:

Screenshot_2020-12-17 Teclado Gamer Fortrek Mecânico K5 RGB Outemu Blue Preto

Acho que deveria ter um min-height e max-width (com width: 100%) maior também

leomp12 commented 3 years ago

https://github.com/ecomplus/storefront/commit/9c088473898da9584b3348e322f2cce549f601a8