Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Add chain_getEvents and chain_proveEvents endpoints #7126

Closed
Tracked by #6950
shuse2 opened this issue Apr 22, 2022 · 0 comments · Fixed by #7159
Closed
Tracked by #6950

Add chain_getEvents and chain_proveEvents endpoints #7126

shuse2 opened this issue Apr 22, 2022 · 0 comments · Fixed by #7159
Assignees
Milestone

Comments

@shuse2
Copy link
Collaborator

shuse2 commented Apr 22, 2022

Description

Add framework endpoints as below

chain_getEvents

Req
{
  height: number,
}
Res
{
  events: {
    moduleID: number,
    typeID: number,
    topics: string[],
    data: string,
    index: number,
  }[],
}

If height does not exist, it should return

{
  error: {
    message: `Events at height ${height} is not stored in the node.`,
  }
}

chain_proveEvents

Req
{
  height: number,
  queries: string[],
}
Res
{
  siblingHashes: string[],
  queries: {
    key: string,
    value: string,
    bitmap: string,
  }[]
}

If height does not exist, it should return

{
  error: {
    message: `Events at height ${height} is not stored in the node.`,
  }
}

Acceptance Criteria

  • All endpoints are unit tested
@shuse2 shuse2 mentioned this issue Apr 22, 2022
4 tasks
@shuse2 shuse2 moved this to Backlog in Lisk SDK Version 6.0.0 Apr 22, 2022
@milenagojkovic milenagojkovic added this to the Sprint 69 milestone May 9, 2022
@shuse2 shuse2 moved this from Backlog to ToDo in Lisk SDK Version 6.0.0 May 9, 2022
@shuse2 shuse2 moved this from ToDo to Pending Review in Lisk SDK Version 6.0.0 May 10, 2022
@shuse2 shuse2 linked a pull request May 10, 2022 that will close this issue
shuse2 added a commit that referenced this issue May 17, 2022
@shuse2 shuse2 closed this as completed May 17, 2022
Repository owner moved this from Pending Review to Done in Lisk SDK Version 6.0.0 May 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants