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

Namespace RPC endpoint and add new endpoints #6991

Closed
Tracked by #6947
shuse2 opened this issue Jan 19, 2022 · 0 comments · Fixed by #7160
Closed
Tracked by #6947

Namespace RPC endpoint and add new endpoints #6991

shuse2 opened this issue Jan 19, 2022 · 0 comments · Fixed by #7160
Assignees
Milestone

Comments

@shuse2
Copy link
Collaborator

shuse2 commented Jan 19, 2022

Description

Current RPC endpoint should be namespaced as below.

chain

  • getBlockByID
  • getBlocksByIDs
  • getBlockByHeight
  • getBlocksByHeightBetween
  • getTransactionByID
  • getTransactionsByIDs
  • getLastBlock

system

  • getNodeInfo
  • getModuleMetadata

generation

  • postTransaction
  • getTransactionsFromPool
  • getStatus (Rename from getForgingStatus)
  • updateStatus (Rename from updateForgingStatus)

Motivation

For version 6, we should reconsider what new endpoints should added for the framework considering new features.
Also, the endpoint should be namespaced to allow more endpoints to be added in the future in organized manner

Acceptance Criteria

  • Update unit tests
  • Each namespace endpoints are in different files
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