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

feat: ship the op roots v1.1 #5248

Merged
merged 532 commits into from
Dec 6, 2023
Merged

feat: ship the op roots v1.1 #5248

merged 532 commits into from
Dec 6, 2023

Conversation

wanglonghong
Copy link
Contributor

@wanglonghong wanglonghong commented Nov 30, 2023 •
edited
Loading

Description

Type of change

  • Docs change / dependency upgrade
  • Configuration / tooling changes
  • Refactoring
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires changes in customer code

High-level change(s) description - from the user's perspective

Related Issue(s)

Fixes

Related pull request(s)

preethamr and others added 30 commits October 12, 2023 17:15
@preethamr
fix: snapshot root db and graph schema changes
314788d
@preethamr
fix: resolve undefined reference
0bacc52
@preethamr
fix: lh propose aggregate root handler
7f255e8
@liu-zhipeng
Merge pull request #5002 from connext/lh_propose_fix
ac4a670 
fix: lh propose aggregate root handler
@preethamr
fix: resolve LH propose task
8dc81c0
@liu-zhipeng
fix: lint
d844a21
@liu-zhipeng
Merge pull request #5011 from connext/propose_to_rootmanager
7e3a86a 
fix: resolve LH propose task
@preethamr
fix: resolve propagate for oproots v1
32e6400
@liu-zhipeng
Merge pull request #5012 from connext/propagate_to_rootmanager
51294c4 
fix: resolve propagate for oproots v1
@preethamr
fix: resolve lh prover for op move v1
8727df9
@preethamr
fix: add lh unit tests
84f0a52
@wanglonghong
fix: lint - unnecessary assertion
bea7274
@wanglonghong
Merge pull request #5014 from connext/prover_op_mode
951667e 
Prover op mode fixes
@carlomazzaferro
fix: sync staging ops with main (#5017)
ecbc10b
@wanglonghong
chore: use the merkletreemanager blocknumber (#5018)
ec8f5c9
@carlomazzaferro
fix: lh propose cron
651b791
@liu-zhipeng
Merge pull request #5019 from connext/fix/re-add-lh-propose
6b6c64e 
fix: lh propose cron
@liu-zhipeng
fix: staging ops
bf2a2dc
@0xng
feat: dispute blocks constructor check [L-1] (#35)
3f72cda
@liu-zhipeng
Merge pull request #5020 from connext/fix_staging_ops
704264d 
fix: staging ops
@0xng
feat: fixing comment [Q-4] (#38)
8d9fe7e
@0xng
feat: removing unused events and errors [Q-2] (#39)
c450171
@0xng
feat: unifying save aggregate root events [Q-3] (#40)
36f18a0
@preethamr
fix: add mq tf config to lh
ad7b9a6
@preethamr
fix: use only primary staging db in lh for now
f097056
@wanglonghong
fix: message queue limit
ff7d48c
@liu-zhipeng
fix: debug custom error hh task
a98fc72
@wanglonghong
fix: message queue config
a45598a 
fix: message queue config
@liu-zhipeng
fix: relayer proxy hub finalize propagate (#5022)
a350212 
* fix: add finalizeAndPropagate func to call from gelato

* fix: cooledDown for propagate
@liu-zhipeng
feat: redeploy staging relayer proxy hub
a288095
@wanglonghong wanglonghong marked this pull request as ready for review December 4, 2023 18:44
LayneHaber
LayneHaber reviewed Dec 5, 2023
View reviewed changes
Copy link
Contributor

@LayneHaber LayneHaber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pretty beefy PR so focused on the contracts package

packages/deployments/contracts/src/cli/init/config/testnet/staging.ts Outdated Show resolved Hide resolved
packages/deployments/contracts/contracts/core/connext/helpers/RelayerProxy.sol Show resolved Hide resolved
packages/deployments/contracts/contracts/core/connext/helpers/RelayerProxy.sol Show resolved Hide resolved
packages/deployments/contracts/contracts/core/connext/helpers/RelayerProxyHub.sol Show resolved Hide resolved
@wanglonghong wanglonghong merged commit 7038c3c into main Dec 6, 2023
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LayneHaber LayneHaber LayneHaber left review comments

@liu-zhipeng liu-zhipeng liu-zhipeng approved these changes

@preethamr preethamr preethamr approved these changes

@just-a-node just-a-node Awaiting requested review from just-a-node just-a-node is a code owner

@prathmeshkhandelwal1 prathmeshkhandelwal1 Awaiting requested review from prathmeshkhandelwal1 prathmeshkhandelwal1 is a code owner

@carlomazzaferro carlomazzaferro Awaiting requested review from carlomazzaferro

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@wanglonghong @preethamr @LayneHaber @liu-zhipeng @carlomazzaferro @0xng @gotzenx