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(blast): add blast info to chain adapters #1652

Merged
merged 14 commits into from
Jul 9, 2024
Merged

feat(blast): add blast info to chain adapters #1652

merged 14 commits into from
Jul 9, 2024

Conversation

bmzig
Copy link
Contributor

@bmzig bmzig commented Jul 1, 2024

This commit should make all necessary changes to the finalizer and relayer so that it can send messages to Blast.

Signed-off-by: bennett <[email protected]>
src/common/Constants.ts Outdated Show resolved Hide resolved
src/common/Constants.ts Outdated Show resolved Hide resolved
@bmzig bmzig marked this pull request as ready for review July 3, 2024 19:15
@bmzig bmzig changed the title feat(blast): support Blast in relayer feat(blast): add blast info to chain adapters Jul 3, 2024
@bmzig
Copy link
Contributor Author

bmzig commented Jul 3, 2024

I don't want this PR to blow up in scope, so we will need to make separate PRs for the finalizer implementation.

src/common/Constants.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@pxrl pxrl left a comment

Choose a reason for hiding this comment

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

LGTM, subject to a couple of minor points.

bmzig and others added 7 commits July 4, 2024 08:34
Signed-off-by: bennett <[email protected]>
…ions (#1662)

The optimism SDK does not have predefined contract addresses for Lisk
like it does for all of our other OP Stack chains. This causes
finalizations to throw undefined errors. This commit adds in the
necessary contract addresses which we then pass into the optimismSDK's
CrossChainMessenger. We will also need to do something similar for
Blast.

---------

Signed-off-by: bennett <[email protected]>
src/clients/bridges/op-stack/OpStackAdapter.ts Outdated Show resolved Hide resolved
@pxrl pxrl merged commit e4b8b51 into master Jul 9, 2024
4 checks passed
@pxrl pxrl deleted the bz/blastInfo branch July 9, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants