pendulum-chain / pendulum-squids

The subsquid squids we use for Pendulum/Amplitude/Foucoco.
GNU General Public License v3.0
0 stars 0 forks source link

refactor storage and event selector #49

Closed gianfra-t closed 7 months ago

gianfra-t commented 7 months ago

This PR is related to the bug described #43 here, and also removes duplicate definition of the event decoder and storage functions.

gianfra-t commented 7 months ago

@pendulum-chain/devs let me know what you think, and then I can try and deploy the squids, and compare some queries.

gianfra-t commented 7 months ago

I saw some errors in foucoco only, from contract events. But I think those were before already.

ebma commented 7 months ago

Depends on the error you saw of course but I think @bogdanS98 also encountered contract errors on Foucoco so it's probably nothing that got introduced in this PR.

gianfra-t commented 7 months ago

Yes I also remember exploring these errors in Foucoco some months ago. Most likely they are the same errors. Since the sync is now complete, I will point to these new squids and stop the old ones.

ebma commented 7 months ago

Alright 👍 ready for merge IMO