Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Fix for missing l2 to l1 messages #330

Merged
merged 3 commits into from
Nov 3, 2022
Merged

Fix for missing l2 to l1 messages #330

merged 3 commits into from
Nov 3, 2022

Conversation

mikiw
Copy link
Contributor

@mikiw mikiw commented Nov 2, 2022 •
edited
Loading

Usage related changes

  • Fix for missing l2 to l1 messages in transaction_receipts

Development related changes

  • test for l2 to l1 messages was added

Checklist:

  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/lint.sh
  • Performed code self-review
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Documented the changes
  • Linked the issues which this PR resolves
  • Updated the tests
  • All tests are passing

@mikiw mikiw linked an issue Nov 2, 2022 that may be closed by this pull request
Missing l2_to_l1_messages #323
Closed
@mikiw mikiw requested a review from FabijanC November 2, 2022 16:08
FabijanC
FabijanC reviewed Nov 3, 2022
View reviewed changes
test/test_postman.py Outdated Show resolved Hide resolved
@mikiw mikiw requested a review from FabijanC November 3, 2022 09:12
@mikiw mikiw merged commit 33ef003 into master Nov 3, 2022
@mikiw mikiw deleted the l2-to-l1-messages-fix branch November 3, 2022 09:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@FabijanC FabijanC FabijanC approved these changes

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

Successfully merging this pull request may close these issues.

Missing l2_to_l1_messages
2 participants
@mikiw @FabijanC