relayer: transaction sender impl #617
Merged
+3,467
−1,575
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
012e6ab to
9927e09
Compare
oclaw
force-pushed
the
feature/relayer-fin-ensurer
branch
2 times, most recently
from
b5ea82f to
07fe050
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
9927e09 to
30fb50a
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
30fb50a to
1c800be
Compare
oclaw
force-pushed
the
feature/relayer-fin-ensurer
branch
from
07fe050 to
121bf84
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
1c800be to
a55bf5a
Compare
oclaw
force-pushed
the
feature/relayer-fin-ensurer
branch
from
121bf84 to
6c1151a
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
a55bf5a to
2d7c288
Compare
oclaw
force-pushed
the
feature/relayer-fin-ensurer
branch
2 times, most recently
from
a24aec9 to
9d2be0d
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
2d7c288 to
fa2012d
Compare
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
7923e85 to
6dd8cb3
Compare
akokoshn
reviewed
Mar 27, 2025
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
6dd8cb3 to
99d0dce
Compare
akokoshn
reviewed
Mar 27, 2025
| rs.Logger.Info().Msg("key generated") | ||
| } | ||
|
|
||
| // TODO deploy relayer's L2 smart account (if it is not exist) |
There was a problem hiding this comment.
I guess here relayer shout check and fails if smart account not exists.
All deployments are CLI tool responsibility
There was a problem hiding this comment.
We did not decide on CLI tool actually, just to make it a common code component :)
Anyway I would like to split smart account related logic from this PR
akokoshn
approved these changes
Mar 27, 2025
relayer: recompile l1/l2 ABI
oclaw
force-pushed
the
feature/relayer-tx-sender
branch
from
20ce142 to
c4c6fb9
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
github-merge-queue bot
pushed a commit
that referenced
this pull request
Mar 31, 2025
relayer: transaction sender impl
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implementation of relayer
TransactionSendercomponentThis component is responsible for:
PR also contains L2 contract wrapper which provides an interface to interact with
L2BridgeMessengercontract and some initial config for it (nil RPC endpoint, contract/account address, key). Current version only generates the key (if it is not exists). Other initialization stuff is going to be implemented in the following PRs