Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assign creatorAddress in handleSubgraphPublishedV2 #192

Conversation

mnlesane
Copy link
Contributor

@mnlesane mnlesane commented Aug 6, 2022

The creatorAddress field has not been assigned for subgraphs created after 2022-02-23, as the graphAccount param is no longer associated with the SubgraphPublished event for v2. This PR uses event.transaction.from instead.

Demonstration:
https://thegraph.com/hosted-service/subgraph/mnlesane/graph-network-mainnet

  subgraphs(orderBy:createdAt, orderDirection:desc
  ) {
    id displayName creatorAddress
  }
}```

@juanmardefago
Copy link
Collaborator

Hey @mnlesane!
Looks good! I'm going to merge it to mainnet-staging now that we got the workflows back to a state where it's only going to be deploying to staing on pushes there.

I'm also gonna merge #182 there, and check if everything works as expected afterwards, before moving it to production!

Thanks a ton for the contributions and sorry for the long delays!

@juanmardefago juanmardefago merged commit 0eccac9 into graphprotocol:mainnet-staging Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants