-
Notifications
You must be signed in to change notification settings - Fork 52
TIP-48: CORE-API #153
base: main
Are you sure you want to change the base?
TIP-48: CORE-API #153
Conversation
TIP-47: Storage Deposit
…on for rewards endpoint.
thibault-martinez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are errors when opening the Core Swagger
* staker: registered validator * validator: committee member
| strongParents: | ||
| - "0x0482f0eba39a23c9a13072c93d828b55543132c47f5f57514d9e55535e9d4f4f35000000" | ||
| - "0xae7c4f55a6db8bf4841e4a38f06d32ab9bd88b927a6ba0bc19bcb19c625ff8b63c000000" | ||
| weakParents: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are not ordered
| blockState: | ||
| type: string | ||
| enum: | ||
| - pending |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to add accepted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't expose accepted to users.
| BlockFullResponse: | ||
| description: Returns the block structure and the metadata of a given block. | ||
| properties: | ||
| block: | ||
| $ref: '#/components/schemas/Block' | ||
| description: The requested block structure. | ||
| metadata: | ||
| $ref: '#/components/schemas/BlockMetadataResponse' | ||
| description: The metadata of the requested block. | ||
| required: | ||
| - block | ||
| - metadata |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be renamed to BlockWithMetadataResponse? Would be consistent with OutputWithMetadataResponse.
| metrics: | ||
| blocksPerSecond: "1.1E+00" | ||
| confirmedBlocksPerSecond: "2.2E+00" | ||
| confirmationRate: "3.3E+00" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really want scientific notation?
| - latestAcceptedBlockSlot | ||
| - latestConfirmedBlockSlot | ||
| - pruningEpoch | ||
| metrics: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be put into its own route + response
| - block | ||
| - metadata | ||
|
|
||
| BlockMetadataResponse: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be updated to latest (and examples)
| - blockId | ||
| - blockState | ||
|
|
||
| TransactionMetadataResponse: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be updated to latest (and examples)
| description: Return the information of committee members at the given epoch index. If epoch index is not provided, the current committee members are returned. | ||
| parameters: | ||
| - in: query | ||
| name: epochIndex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| name: epochIndex | |
| name: epoch |
Swagger Previews:
Note that this TIP is in Draft mode and as such is still being updated.