aptos-core icon indicating copy to clipboard operation
aptos-core copied to clipboard

[RFE] Improve Staking documentation, explain configuration

Open clay-aptos opened this issue 2 years ago • 7 comments

🐛 Request for Enhancement

Regarding: https://aptos.dev/concepts/staking/#staking-on-the-aptos-blockchain

We should include the data that's currently on chain and a pointer to it. Specifically: https://mainnet.aptoslabs.com/v1/accounts/0x1/resource/0x1::staking_config::StakingConfig That points to: https://github.com/aptos-labs/aptos-core/blob/main/aptos-move/framework/aptos-framework/sources/configs/staking_config.move

We should also interpret the parameters in our tech doc too.

Finally, I see now this link in the doc:

Triggers at the epoch start
TIP
See https://github.com/aptos-labs/aptos-core/blob/0daade4f734d1ba29a896b00d7ddde2249e87970/aptos-move/framework/aptos-framework/sources/configs/stake.move#L862 for the full code.

But I think that might be an older version of the staking_config.move file above. If so, we should re-link and make clear the purpose of that file.

clay-aptos avatar Oct 31 '22 16:10 clay-aptos

Should we track Aptos Governance in a separate issue?

movekevin avatar Nov 01 '22 17:11 movekevin

Should we track Aptos Governance in a separate issue?

Yes, removed from here. Want me to file the separate issue? What are we looking to document in governance?

clay-aptos avatar Nov 01 '22 18:11 clay-aptos

Should we track Aptos Governance in a separate issue?

Yes, removed from here. Want me to file the separate issue? What are we looking to document in governance?

Filed but devoid of details: https://github.com/aptos-labs/aptos-core/issues/5399

Please add them.

clay-aptos avatar Nov 01 '22 20:11 clay-aptos

Hi @movekevin and @davidiw , please see: https://github.com/aptos-labs/aptos-core/pull/5401

clay-aptos avatar Nov 01 '22 21:11 clay-aptos

I also saw this referenced in a thread about request_commissions: https://github.com/aptos-labs/aptos-core/blob/30b385bf38d3dc8c4e8ee0ff045bc5d0d2f67a85/aptos-move/framework/aptos-framework/sources/staking_contract.move

Should we point to all of the stak*.move files in this directory for more information. https://github.com/aptos-labs/aptos-core/tree/30b385bf38d3dc8c4e8ee0ff045bc5d0d2f67a85/aptos-move/framework/aptos-framework/sources

clay-aptos avatar Nov 03 '22 16:11 clay-aptos

And link to: https://aptos.dev/nodes/validator-node/operator/staking-pool-operations/#requesting-commission

clay-aptos avatar Nov 03 '22 16:11 clay-aptos

https://aptos.dev/nodes/validator-node/operator/staking-pool-operations/#description-of-output-fields

clay-aptos avatar Nov 03 '22 16:11 clay-aptos

@movekevin , please reopen this if you have more planned for this topic soon. Thanks!

clay-aptos avatar Nov 07 '22 16:11 clay-aptos