Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make custom Validation/ExecutionContext under ICS-07 portable #1163

Closed
Tracked by #1164
Farhad-Shabani opened this issue Apr 15, 2024 · 0 comments · Fixed by #1166
Closed
Tracked by #1164

Make custom Validation/ExecutionContext under ICS-07 portable #1163

Farhad-Shabani opened this issue Apr 15, 2024 · 0 comments · Fixed by #1166
Assignees
Labels
O: usability Objective: aims to enhance user experience (UX) and streamline product usability

Comments

@Farhad-Shabani
Copy link
Member

Feature Summary

The context.rs module under the ICS-07 contains a set of custom Validation/ExecutionContext traits, currently tightly coupled with ConsensusStateType specific to Tendermint light client implementation. However, their potential extends beyond Tendermint to benefit other light client implementations, including Sovereign's, hosted in the sovereign-ibc repository.
Although initially designed for ICS-07, the methods within these traits are sufficiently general to be valuable for a wider range of light clients.

Proposal

To enhance their usability across diverse light client implementations, we propose relocating these traits from ICS-07 to ICS-02, aligning them with other client context traits. They would serve as optional traits, enriching client capabilities in state validation. This adjustment aims to foster code reusability and contribute to a universal library for implementing clients as CosmWasm contracts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O: usability Objective: aims to enhance user experience (UX) and streamline product usability
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant