Skip to content

Consider renaming Event.logIndex to Event.blockLogIndex #2038

@PaulRBerg

Description

@PaulRBerg

Which packages are impacted by your issue?

@graphprotocol/graph-ts

Describe the issue

Problem

When looking at these types:

public logIndex: BigInt,
public transactionLogIndex: BigInt,

It's not obvious that the logIndex is the index of the event in the entire block, whereas transactionLogIndex is the index of the event in the transaction.

Confused Users

Solution

  • Rename Event.logIndex to Event.blockLogIndex.
  • For backward-compatibility, maintain Event.logIndex, but annotate it as deprecated.

Related

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions