Subgraph: Fix withdrawal finalization support #562
Assignees
Labels
Comments
This was referenced Jul 5, 2024
Merged
kkosiorowska
added a commit
that referenced
this issue
Jul 9, 2024
This PR adds support for indexing the withdrawals in Acre. Here we subscribe to the `RedemptionRequested` event from the tBTC Bridge contract. Because we are listening to events from tBTC bridge not from `BitcoinRedeemer` we must make sure the withdrawal comes from the Acre network (the Acre subgraph should index only Acre withdrawals). Also, t's possible to create 2 redemptions with the same redemption key in tBTC v2 network (once the previous ones are finalized- meaning only one pending redemption with the same key can exist). In that case we must add a suffix to the `Withdraw` id - we count redemptions with the same redemption key in `RedemptionKeyCounter` entity. Currently, we do not support the withdrawals finalization - I need to investigate why the wrong redemption key is built based on the parameters from the `submitRedemptionProof` call. This will be handled in #562 Closes: #276 Closes: #268
nkuba
added
🔌 SDK
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As mentioned in #513:
We must figure out what's wrong and fully support it in the Acre subgraph.
Once solved, we need to remove the Threshold subgraph reference in SDK.
The text was updated successfully, but these errors were encountered: