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

Read bulk_payout logs to avoid double payouts #434

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alidzm
Copy link
Collaborator

@alidzm alidzm commented Nov 29, 2022

To avoid double payouts bulk_payout method has been reworked: read bulk_payout transaction receipt and check events of transaction to catch Transfer event and make sure that payout has been processed.

@alidzm alidzm requested a review from igervasiychuk November 29, 2022 09:30
hmt_escrow/job.py Outdated Show resolved Hide resolved
hmt_escrow/job.py Show resolved Hide resolved
hmt_escrow/job.py Show resolved Hide resolved
hmt_escrow/job.py Show resolved Hide resolved
@alidzm alidzm force-pushed the prevent_double_payouts branch 4 times, most recently from 7b1f387 to 27c006e Compare November 30, 2022 06:30
hmt_escrow/eth_bridge.py Outdated Show resolved Hide resolved
hmt_escrow/eth_bridge.py Outdated Show resolved Hide resolved
hmt_escrow/eth_bridge.py Outdated Show resolved Hide resolved
hmt_escrow/eth_bridge.py Outdated Show resolved Hide resolved
hmt_escrow/job.py Outdated Show resolved Hide resolved
hmt_escrow/eth_bridge.py Outdated Show resolved Hide resolved
@alidzm alidzm force-pushed the prevent_double_payouts branch from 27c006e to eccf82a Compare November 30, 2022 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants