Making the timestamp available to the rms on commits - Githubissues

osmosis-labs / cosmos-sdk

:chains: A Framework for Building High Value Public Blockchains :sparkles:

https://cosmos.network/

Apache License 2.0

14 stars 31 forks source link

Making the timestamp available to the rms on commits #485

Closed nicolaslara closed 11 months ago

nicolaslara commented 11 months ago

Ref: https://github.com/osmosis-labs/osmosis/issues/5960

What is the purpose of the change

Historical wasm queries depend on the block timestamp being available on the queryContext (https://github.com/osmosis-labs/wasmd/blob/master/x/wasm/types/types.go#L273-L277). The timestamp is added to the context in https://github.com/osmosis-labs/cosmos-sdk/blob/osmosis-main/baseapp/abci.go#L685-L693, but that doesn't work on 0.45 because the timestamp is never stored.

This PR fixes that.

Brief Changelog

Store the timestamp in the RootMultiStore like it's done in the latest version of the SDK

Testing and Verifying

all tests should pass

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable / specification (x/<module>/spec/) / Osmosis docs repo / not documented)