Remove Context.BlockHeight #4578
Labels
good first issue
T: Dev UX
UX for SDK developers (i.e. how to call our code)
Type: Code Hygiene
General cleanup and restructuring of code to provide clarity, flexibility, and modularity.
Summary
The SDK's
Context
type provides the ability to set/get the following:All of these (excluding the header) are part of the header itself.
Problem Definition
When setting the block height, proposer or time, the header is updated accordingly but not the inverse! This can lead to bugs and a poor developer UX.
Proposal
Keep the functionality of allowing the developer to set (
With*
) block fields, but only allow getting the header. i.e. removeBlockHeight
For Admin Use
The text was updated successfully, but these errors were encountered: