Skip to content

Conversation

@RomanHodulak
Copy link
Contributor

@RomanHodulak RomanHodulak commented Oct 26, 2024

Ticket

What does this PR do?

  • Summary: Implements all needed changes to make sending layer-2 transactions possible.
  • Key Changes:
    • Setup RPC client for building transactions
    • Fills L2 transaction with values
    • Changes contract calls to send transaction and wait

Does this PR introduce any breaking changes (API/schema)?

  • No

Do any environment variables need to be updated or added before deployment?

  • No

How can this PR be tested?

In one terminal, run the sequencer

just -f .devcontainer/justfile op-up
just -f .devcontainer/justfile op-update-chain-address
cargo make interceptor

In another terminal, send transaction

curl -X POST http://localhost:8456 --header 'Content-Type: application/json' --data '{"jsonrpc":"2.0","method":"eth_sendRawTransaction","params":["0xb85902f85682038501808088ffffffffffffffff808080c001a0d555dc3a308d5bde3d5bc665796f9e7d7125c1554667325533fe237c1aa120b5a07d97dae06082d3eb7fa8966b33f6ce51d7127dcddd5da3d8be9c448a72150a90"],"id":1}'

You will get a response containing L2 transaction hash.

@linear
Copy link

linear bot commented Oct 26, 2024

SEQ-210 Integration testing via dev container

When SEQ-175 is mostly complete and the full stack is stood up with all parts communicating successfully, we need to test that the sequencer's send_raw_transaction() flow.

AC

  • Stand up dev container and submit an L3 transaction to the sequencer. Transaction should be forwarded to the L2 smart contract
    • Variations on the L3 txn including edge cases and large ones that need to be chunked up
  • (Nice to have) Crystallize the above with a CI test - dependent on SEQ-140

@RomanHodulak RomanHodulak force-pushed the roman/SEQ-210/send-l2-tx branch from aab4260 to d320320 Compare October 26, 2024 15:42
@RomanHodulak RomanHodulak changed the base branch from main to roman/SEQ-210/localnet-setup October 26, 2024 15:42
@RomanHodulak RomanHodulak changed the base branch from roman/SEQ-210/localnet-setup to roman/SEQ-249/localnet-setup October 26, 2024 15:57
Copy link
Collaborator

@daniilrrr daniilrrr left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Contributor

@WillPapper WillPapper left a comment

Choose a reason for hiding this comment

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

Thank you @RomanHodulak! This looks great

@RomanHodulak RomanHodulak force-pushed the roman/SEQ-249/localnet-setup branch from dd8476c to 6bfef95 Compare October 29, 2024 14:58
Base automatically changed from roman/SEQ-249/localnet-setup to main October 29, 2024 15:00
@RomanHodulak RomanHodulak force-pushed the roman/SEQ-210/send-l2-tx branch from d320320 to b5f7b68 Compare October 29, 2024 15:41
@RomanHodulak RomanHodulak merged commit 8d1e7b9 into main Oct 29, 2024
2 checks passed
@RomanHodulak RomanHodulak deleted the roman/SEQ-210/send-l2-tx branch October 29, 2024 15:44
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.

4 participants