Skip to content

sc: transfer bridge data to NillRollup #905

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
Open
wants to merge 1 commit into
base: feature/nilrollup-deposit-count-enhancement
Choose a base branch
from
Open

sc: transfer bridge data to NillRollup #905

wants to merge 1 commit into from

Conversation

oclaw
Copy link
Contributor

@oclaw oclaw commented May 16, 2025 •
edited
Loading

Short Summary

Adjusted sync committee proposer in order to pass bridge-related data to the NilRollup contract

What Changes Were Made

  • Got rid of hardcoded ABI in favor of generating it from actual solidity contracts
  • Replaced MessageCount field with DepositNonce one
  • Implemented BridgeStateGetter for fetching data from the latest main shard block of the batch

Checklist

  • I have read and followed the Contributing Guide
  • I have tested the changes locally
  • I have added relevant tests (if applicable)
  • I have updated documentation/comments (if applicable)

@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from adf608d to b4d5d24 Compare May 16, 2025 10:48
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from b4d5d24 to 092d45b Compare May 16, 2025 11:02
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from 092d45b to f156d0c Compare May 16, 2025 11:13
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from f156d0c to 5550baf Compare May 16, 2025 11:17
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from 5550baf to 0443d3f Compare May 16, 2025 11:23
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from 0443d3f to ff9370a Compare May 16, 2025 13:07
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from ff9370a to 5024295 Compare May 16, 2025 13:28
@oclaw oclaw force-pushed the feature/rollup-contract-adjustment branch from 5024295 to 412e746 Compare May 16, 2025 14:12
@oclaw oclaw marked this pull request as ready for review May 16, 2025 15:37
@oclaw oclaw self-assigned this May 16, 2025
@oclaw oclaw requested review from zadykian, defistar and akokoshn May 16, 2025 15:46
@notion-workspace Notion Workspace
Copy link

Send bridge related data to NilRollUp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zadykian zadykian Awaiting requested review from zadykian

@defistar defistar Awaiting requested review from defistar

@akokoshn akokoshn Awaiting requested review from akokoshn

Assignees

@oclaw oclaw

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@oclaw