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

feat: add IBC prefix for ics-08 clients and bugfixes #826

Conversation

AntonAndell
Copy link
Collaborator

@AntonAndell AntonAndell commented Jan 23, 2024

Description

Commit Message

type: commit message

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

Checklist

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines
  • I have added version bump label on PR.

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Add revision number but for now do not add support for resseting chain height
@AntonAndell AntonAndell force-pushed the 808-investigate-commitment-prefix-usage-in-ibc-java branch from 3f0c1e5 to 2a6b7d7 Compare January 23, 2024 09:28
@AntonAndell AntonAndell added the enhancement New feature or request label Jan 23, 2024
@AntonAndell AntonAndell force-pushed the 808-investigate-commitment-prefix-usage-in-ibc-java branch from 2a6b7d7 to ec773cf Compare January 23, 2024 09:36
@AntonAndell AntonAndell force-pushed the 808-investigate-commitment-prefix-usage-in-ibc-java branch from 537cfa2 to 0cb76eb Compare February 7, 2024 05:38
@AntonAndell AntonAndell force-pushed the 807-add-client-specific-hashing-on-ack-and-packet-data branch 2 times, most recently from dc07e0d to 7d0585a Compare February 12, 2024 10:42
…on-ack-and-packet-data' into 808-investigate-commitment-prefix-usage-in-ibc-java
@AntonAndell AntonAndell merged commit a7af666 into 807-add-client-specific-hashing-on-ack-and-packet-data Feb 12, 2024
4 checks passed
@AntonAndell AntonAndell deleted the 808-investigate-commitment-prefix-usage-in-ibc-java branch February 12, 2024 10:54
AntonAndell added a commit that referenced this pull request Feb 12, 2024
* feat: hash depending on client type

* feat: add IBC prefix for ics-08 clients and bugfixes (#826)

* feat: Add revision number to light client heights

Add revision number but for now do not add support for resseting chain height

* revert to using two clients

* feat: add IBC prefix for ics-08 clients and bugfixes

* fix: hash ack value before sending to verify membership for ics8 client

---------

Co-authored-by: izyak <[email protected]>

---------

Co-authored-by: izyak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants