docs: add tx filterer docs, fix minor mistakes #996
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
|
Visit the preview URL for this PR (updated for commit f917710): https://staging-zksync-web-era-docs--pr996-add-tx-filterer-doc-c6pcuobu.web.app (expires Thu, 16 May 2024 13:05:14 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: f408c7165e11221d785cc3915b60f2a4a4f277f9 |
shahar4
reviewed
Mar 29, 2024
MexicanAce
reviewed
Mar 30, 2024
|
|
||
| A hyperchain operator can filter L1→L2 transactions coming through the Diamond proxy. It is achieved by having an | ||
| address of the special contract, responsible for transaction filtering, in the state transition storage. If the filterer | ||
| exists, it is being called in the [Mailbox facet](https://github.com/matter-labs/era-contracts/blob/dev/l1-contracts/contracts/state-transition/chain-deps/facets/Mailbox.sol#L243) |
There was a problem hiding this comment.
These links are for the dev branch. As these are not live, do we want to hold off on merging these changes until 1.5.0 is live?
There was a problem hiding this comment.
Yeah, it makes sense to wait for 1.5.0 before merging. I'll keep track of it and ping you when it's ready to be merged
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.
What 💻
Why ✋