BM-1603: Add povw/staking/delegation API #1138
Conversation
willpote
requested review from
a team,
Wollac,
austinabell,
capossele,
willemolding and
zeroecco
as code owners
github-actions
bot
changed the title
| pub use get_rewards_delegates::{get_rewards_delegates, ZkcGetRewardsDelegates}; | ||
| pub use get_staked_amount::{get_staked_amount, ZkcGetStakedAmount}; | ||
| pub use stake::ZkcStake; | ||
| pub use summary::ZkcSummary; |
There was a problem hiding this comment.
I think we should maybe hide this one behind a experimental or unstable feature?
There was a problem hiding this comment.
This is a highly requested feature, and would be one of the most used commands (as it can be used multiple times an epoch across many epochs). I would recommend, if we can, to make this feature stable asap; see my comment below.
There was a problem hiding this comment.
This is a highly requested feature, and would be one of the most used commands (as it can be used multiple times an epoch across many epochs). I would recommend, if we can, to make this feature stable asap; see my comment below.
Agree, looking forward to this, urgently needed.
|
This is really helpful @willpote; I have had exact feedback asking for this CLI feature many times over the past 48 hours. I would suggest we make this feature live as soon as we can; this will really help provers understand and therefore help faciliate participation in the market and PoVW. I wouldn't mark it as experimental, the only other option really is a web UI where provers can connect various addresses to see this visually, which has also been requested. If we do decide to go that route, that would take some time, and regardless there are always provers who prefer CLI commands like this for scripting reasons etc. |
Sounds good. Will try and get it stable today. |
willpote
changed the title
willpote
changed the title
| { | ||
| Effect: 'Allow', | ||
| Action: ['logs:CreateLogStream', 'logs:PutLogEvents'], | ||
| Resource: '*', |
There was a problem hiding this comment.
we need to find a way to neck this down.
There was a problem hiding this comment.
Refactored so we create the Cloudwatch policy in the Market indexer + Reward indexer stack themselves, rather than this shared infra stack, so I can get the log group names
|
🚀 Documentation Preview Deployment URL: https://boundless-documentation-1yku1157d-boundless-network.vercel.app Updated at: 2025-10-02 18:06:48 UTC |
Indexes rewards information and serves the data via an API. Will be integrated into the explorer and the CLI. To do this I:
market-indexerandreward-indexer, conditionally based on the Pulumi config. E.g. on Eth Mainnet, we don't have a Boundless market, so we only deploy thereward-indexerinPulumi.config.1.yaml