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

Use universal client ids #258

Closed
srdtrk opened this issue Feb 6, 2025 · 0 comments · Fixed by #262
Closed

Use universal client ids #258

srdtrk opened this issue Feb 6, 2025 · 0 comments · Fixed by #262
Assignees
Labels
nice to have solidity Issues related to the solidity contracts

Comments

@srdtrk
Copy link
Member

srdtrk commented Feb 6, 2025

Currently, this implementation supports light client types such as 07-tenderemint. This means all our light client id's are prefixed by these types such as 07-tendermint-0. This makes it harder to iterate over all light clients. We should use a universal ids so that client ids are client-0, client-1, ...

We should also make sure that a user can iterate over them by having public access to the latest client number.

@srdtrk srdtrk added nice to have solidity Issues related to the solidity contracts labels Feb 6, 2025
@github-project-automation github-project-automation bot moved this to Backlog in IBC-GO Eureka Feb 6, 2025
@gjermundgaraba gjermundgaraba assigned srdtrk and unassigned srdtrk Feb 6, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in IBC-GO Eureka Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nice to have solidity Issues related to the solidity contracts
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant