Skip to content
This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Introduce TransactionReceipt and Log #260

Merged
merged 4 commits into from
Mar 30, 2022
Merged

Conversation

tilacog
Copy link
Contributor

@tilacog tilacog commented Mar 21, 2022

This PR introduces two new Ethereum types:

  • TransactionReceipt
  • Log

Since apiVersion 0.0.7, subgraphs can request transaction receipts inside event handlers, so, for now, TransactionReceipt is stored in Event.receipt (which is a nullable field).

This also bumps the npm version to 0.27.0


This PR is a dependency of:

chain/ethereum.ts Outdated Show resolved Hide resolved
@neysofu
Copy link
Member

neysofu commented Mar 28, 2022

Do you need another review? @tilacog

@tilacog
Copy link
Contributor Author

tilacog commented Mar 28, 2022

Do you need another review? @tilacog

If it's Ok for you, it would be awesome! 😁

Copy link
Contributor

@evaporei evaporei left a comment

Choose a reason for hiding this comment

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

I've commented on the graph-node PR about a couple of field names that might require change.

Also I think you need to add the Index variants here.

@tilacog tilacog force-pushed the tiago/receipts-in-mappings branch from 469e308 to d8a5602 Compare March 29, 2022 21:28
@tilacog
Copy link
Contributor Author

tilacog commented Mar 29, 2022

Added a couple of PR's with the requested changes.

chain/ethereum.ts Outdated Show resolved Hide resolved
@tilacog tilacog force-pushed the tiago/receipts-in-mappings branch from 8141b03 to b1be084 Compare March 30, 2022 23:27
@tilacog tilacog requested a review from evaporei March 30, 2022 23:29
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.

3 participants