search

cowprotocol / subgraph

CoW Protocol Subgraph
4 stars 5 forks source link

Deploy a POC based on the workaround E&N Provided. #60

Open GabrielCamba opened 2 years ago

GabrielCamba commented 2 years ago

We need to index the settle function call handler for being able to decode the AppData.

I faced some issues trying to index that function. I arrived to an error reported 3 years ago. The error is because of a multidimensional array that is not currently supported by The Graph. We have a parameter that is tuple[][3] that is not supported. That error prevents the contract of being indexed, because of an error when graph codegen statement is executed. I read the workaround for that and seems to be replacing that tuple multi array and left only a single dimension of it such as: tuple[] That works and allow us to keep working on the subgraph and indexing the events.

When back to index the settle function call there's no error but nothing was indexed. I isolate the issue and redeployed for being able to share the error with E&N people.

Krishnanand V P from E&N replied on slack a workaround for being able to index the first part of the parameters until they provide a final solution they're working on.

What he suggested is to change from tuple[][3] to bytes[3] that will index that function call.

image.png

GabrielCamba commented 2 years ago

this morning I deployed that fix here: https://thegraph.com/hosted-service/subgraph/gabrielcamba/gpv2poc?version=pending So far no calls were counted.

IF it finishes properly and start counting the calls, let's move that basic implementation to the main subgraph to start building on top of that.