Consensys / ethql

A GraphQL interface to Ethereum :fire:
Apache License 2.0
623 stars 85 forks source link

Update to only fetch transactions for blocks when necessary. #50

Closed akhila-raju closed 6 years ago

akhila-raju commented 6 years ago

Implements #49.

raulk commented 6 years ago

@akhila-raju a few comments:

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

raulk commented 6 years ago

@akhila-raju pushed a few commits here to:

Further refactoring is required in the model, as it's questionable for this logic to be in the EthqlQuery root query, it should be in the EthqlBlock class so it can be reused across other fields that yield a Block (parent, ommers, and others down the line).