Skip to content

indexer: update finalizations#3324

Merged
mslipper merged 1 commit intodevelopfrom
tux/feat/indexer-upgrade-bedrock
Aug 26, 2022
Merged

indexer: update finalizations#3324
mslipper merged 1 commit intodevelopfrom
tux/feat/indexer-upgrade-bedrock

Conversation

@tuxcanfly
Copy link
Contributor

@tuxcanfly tuxcanfly commented Aug 26, 2022

Description
This PR updates the indexer to track deposit / withdrawal finalisations.

Additional context
The L1 / L2 services are updated to listen to DepositFinalized on L2 and ETHWithdrawalFinalized / ERC20WithdrawalFinalized on L1 respectively.

The /withdrawal/<address> API endpoint has been updated to return finalisation status including the L1 block hash / number at which it finalised.

Metadata

@changeset-bot
Copy link

changeset-bot bot commented Aug 26, 2022

⚠️ No Changeset found

Latest commit: 2414056

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mergify
Copy link
Contributor

mergify bot commented Aug 26, 2022

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

@tuxcanfly tuxcanfly force-pushed the tux/feat/indexer-upgrade-bedrock branch from 25b80cd to 61a2771 Compare August 26, 2022 22:40
@tuxcanfly tuxcanfly force-pushed the tux/feat/indexer-upgrade-bedrock branch from 61a2771 to 2414056 Compare August 26, 2022 22:45
@mslipper mslipper merged commit 0230302 into develop Aug 26, 2022
@mslipper mslipper deleted the tux/feat/indexer-upgrade-bedrock branch August 26, 2022 23:21
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