Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Conversation

@Seb-figment
Copy link

No description provided.

Comment on lines 16 to 17
"@graphprotocol/graph-cli": "https://github.com/figment-networks/graph-cli.git#cosmos-v1.1",
"@graphprotocol/graph-ts": "https://github.com/figment-networks/graph-ts.git#cosmos-v1.1"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use the official packages here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

messageHandlers is merged upstream ?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If not, we should do that before releasing this example.

Comment on lines 45 to 52
function validateBigInt(str: string): boolean
{
for(let i = 0; i < str.length; i++) {
const num = parseInt(str[i])
if (num == NaN) {
return false;
}
};
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could use the built-in API for that? https://thegraph.com/docs/en/developer/assemblyscript-api/#big-int=

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The BigInt.fromString constructor fail if the string is not all digit. I've seen occurence where the string is "Nan"

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it, let's keep your version then.

},
"dependencies": {
"@graphprotocol/juno-ts": "https://github.com/figment-networks/graph-juno-ts.git",
"assemblyscript-json": "^1.1.0",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can remove that now.

@@ -0,0 +1,3 @@
# CosmWasm Liquidity Example

This example subgraph stores `Contract` objects that represent CosmWasm contract. In order to do that, the handler function decode all `MsgExecuteContract` messages.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the handler function decode all MsgExecuteContract messages.

decode => decodes

@sascha1337
Copy link

so?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants