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.

Sign up for GitHub

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

Jump to bottom

Contract updates addressing the ibc-go wasm light client refactoring #388

Merged
merged 47 commits into from
Dec 15, 2023
Merged

Contract updates addressing the ibc-go wasm light client refactoring #388

merged 47 commits into from
Dec 15, 2023

Conversation

misko9
Copy link
Contributor

@misko9 misko9 commented Aug 7, 2023

No description provided.

vmarkushin and others added 26 commits July 20, 2023 22:32
@vmarkushin
Pass ConsensusState and ClientState in contract initialization method
f2f6a18
@vmarkushin @misko9
wip
767e060
@misko9
Change code_id to code_hash
19759a6
@misko9
Make executes -> sudo, move some execute msgs to query, add read-only
d3e49b4 
context for queries, and add TimestampAtHeight query.
@misko9
Merge wasm header and misbehaviour to Client Message
0273593
@misko9
Return heights on update state
98c6294
@misko9
Re-use context instead of adding a context for read only
e8b24e2
@misko9
clean up imports
f65155a
@misko9
Update grandpa
7b25870
@misko9
Get hypserspace to build
eb6cfba
@misko9
Merge pull request #1 from misko9/steve/contracts2
9717315 
Update grandpa and get hyperspace to build
@misko9
E2e wip
d5ec645
@misko9
Update client message
fce2d63
@misko9
Revert parachain version
4b2fa70
@misko9
Add parachain user to parachain docker file
3de0e2a
@crodriguezvega
imp: remove unused fields in contract result
6f36ad2
@crodriguezvega
fix: make verify (non) membership sudo calls
7bae68e
@misko9
Merge pull request #3 from crodriguezvega/carlos/sudo-verify-(non)-me…
91fd02a 
…mbership

imp: sudo verify (non) membership
@misko9
Merge pull request #2 from crodriguezvega/carlos/remove-unused-fields…
dbb69e8 
…-contract-result

imp: remove unused fields in contract result
@misko9
Change QueryResult to ContractResult for Verify(Non)Memberships
4a89080
@crodriguezvega
imp: rename checksubstituteandupdatestate to migrateclientstore
2b6fd6a
@blasrodri
ics10-grandpa-cw: decode consensus state to check for expiration
35607f5
@misko9
Merge pull request #5 from crodriguezvega/carlos/refactor-to-migrate-…
c50ff72 
…client-store

imp: rename `CheckSubstituteAndUpdateState` to `MigrateClientStore`
@blasrodri
compare in seconds
fe0470f
@blasrodri
add test
8b458d1
@misko9
Merge branch 'master' into steve/contracts
44a565c
@misko9 misko9 marked this pull request as ready for review November 7, 2023 18:16
vmarkushin
vmarkushin reviewed Nov 27, 2023
View reviewed changes
Copy link
Collaborator

@vmarkushin vmarkushin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great PR! Thank you @misko9

light-clients/ics07-tendermint-cw/src/context.rs Outdated Show resolved Hide resolved
light-clients/ics10-grandpa-cw/src/context.rs Outdated Show resolved Hide resolved
@vmarkushin vmarkushin enabled auto-merge (squash) December 15, 2023 20:54
@vmarkushin vmarkushin merged commit 5fd0719 into ComposableFi:master Dec 15, 2023
2 checks passed
dhruvja added a commit that referenced this pull request Mar 13, 2024
@dhruvja
Revert "Contract updates addressing the ibc-go wasm light client refa…
1ecff56 
…ctoring (#388)"

This reverts commit 5fd0719.
vmarkushin pushed a commit that referenced this pull request Jun 3, 2024
@misko9 @vmarkushin
Contract updates addressing the ibc-go wasm light client refactoring (#…
20891ca 
…388)

(cherry picked from commit 5fd0719)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charleenfei charleenfei charleenfei left review comments

@vmarkushin vmarkushin vmarkushin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@misko9 @vmarkushin @charleenfei @crodriguezvega @blasrodri