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

Update types foucoco amplitude v0942 #61

Closed gianfra-t closed 5 months ago

gianfra-t commented 6 months ago

Closes #60.

Steps

Note: all new squid versions should be bumped to v20.

ebma commented 6 months ago

Nice 👍 can we merge https://github.com/pendulum-chain/pendulum-squids/pull/59 first? If we deploy these squids without it, they would again miss the await statements and I'll get spammed with emails that our squid missed to wait for database operations.

gianfra-t commented 6 months ago

Sure you are right. I will merge it but only bump the version here if that's okay.

ebma commented 6 months ago

Sure, let's merge it into this one and go for v20 👍

gianfra-t commented 6 months ago

This is an odd case because the Pendulum squid should also be bumped to v20 since it has some changes, yet we cannot upgrade the types still. Upgrading it to v20 now would be confusing since it would be 1 version ahead later.

Perhaps we could leave this PR open until we can upgrade the types for Pendulum and deploy all of them. In the meantime, I deployed Amplitude/Foucoco.

ebma commented 6 months ago

Perhaps we could leave this PR open until we can upgrade the types for Pendulum and deploy all of them.

Good idea 👍 To make it more transparent please add a new TODO checklist item to the description of the PR that mentions we still need to update the types on Pendulum.