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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions