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

Store chain ID in both Starknet and Comet client state #147

Open
Tracked by #123
soareschen opened this issue Dec 17, 2024 · 0 comments · May be fixed by #173
Open
Tracked by #123

Store chain ID in both Starknet and Comet client state #147

soareschen opened this issue Dec 17, 2024 · 0 comments · May be fixed by #173
Assignees
@ljoss17
Labels
cairo comet light client starknet light client
Milestone
MVP Demo #2

Comments

@soareschen
Copy link
Collaborator

soareschen commented Dec 17, 2024 •
edited
Loading

The client states should contain the chain ID.

After updating the fields, the ClientStateFieldsGetter implementation should be updated accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljoss17 ljoss17

Labels
cairo comet light client starknet light client
Projects
None yet
Milestone
MVP Demo #2
Development

Successfully merging a pull request may close this issue.

Add ChainId to Starknet and Comet client state informalsystems/ibc-starknet
4 participants
@soareschen @rnbguy @ljoss17 @Farhad-Shabani