Pedro Pablo Aste Kompen
Pedro Pablo Aste Kompen
This is the rudest pull request I've ever seen so far.
Hola @JJJGGG, hasta ahora nos habían reportado esto en Chrome y un caso en Edge, pero nunca en Firefox 😱 Podrías intentar desactivar y activar (sin eliminar) la extensión y...
No, nunca lo he podido reproducir.
Going to switch to english here, * **Issue**: storage is cleared (apparently randomly) for some users By looking at the error I suspect this might be related to webext-redux `wrapStore`....
Hi @riadeane, no updates, I still can not reproduce it 😓
La versión de la tienda de opera está desactualizada, el proceso de actualización es más tedioso y conlleva varios rechazos antes de poder actualizar, por eso se prefiere la versión...
Hey @MetaMask/confirmations-system-team, your review is the last one missing to merge this PR please 🙏
I'm concerned about the number of test ids being added, given that we know the default values, would not it be better to assert the presence of the default values...
I have one improvement we could use in favor of the new `useLayoutEffect`, should this be addressed in a new PR? Improvement diff ```diff diff --git a/app/components/UI/Ramp/Aggregator/components/Quote/Quote.tsx b/app/components/UI/Ramp/Aggregator/components/Quote/Quote.tsx index 4b09c987b9..c2ac4ef22a...
> But I would recommend doing it in a new PR just after we merge the new arch, just to keep the scope of this PR focused. Is that ok...