Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

fix: bring back access to rollup's current time & height + update Sovereign dependencies to rev cf048cd59 #105

Merged
merged 4 commits into from
Mar 25, 2024
Merged

fix: bring back access to rollup's current time & height + update Sovereign dependencies to rev cf048cd59 #105

merged 4 commits into from
Mar 25, 2024

Conversation

Farhad-Shabani
Copy link
Member

@Farhad-Shabani Farhad-Shabani commented Mar 25, 2024 •
edited
Loading

Closes: #66

Description

  • Removed reliance on the generic DA: DaSpec in IBC modules.
  • Introduced new fields: host_height_map and host_timestamp_map.
  • Updated host relevant information (height, timestamp, ConsensusState) via the sov-consensus-state-tracker kernel module, eliminating the need for the sov-chain-state module in sov-ibc.
  • Removed the context field from IbcContext as host height and timestamp are now defined directly under sov-ibc.
  • Brought back test_create_client_on_sov and test_mint_burn_on_sov tests.
  • Updated Sovereign SDK dependencies to revision cf048cd59

PR author checklist

  • Linked to GitHub issue.
  • Added tests.
  • Updated code comments and documentation (e.g., docs/).

@Farhad-Shabani Farhad-Shabani marked this pull request as ready for review March 25, 2024 03:11
@Farhad-Shabani Farhad-Shabani merged commit 0c3b99f into main Mar 25, 2024
10 checks passed
@Farhad-Shabani Farhad-Shabani deleted the farhad/access-rollup-height-time branch March 25, 2024 03:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
sovereign-ibc
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Can't access rollup's current height and time in RPC methods
1 participant
@Farhad-Shabani