Closed ekzyis closed 5 months ago
worker/wallet.js (2)
`241-245`: Ensure that the logging for autowithdrawals is triggered only under the correct conditions to prevent duplicates. --- `263-271`: Verify the integration and error handling of the `reverse_withdrawl` SQL function within the JavaScript logic.
Description
A slice of my logs:
This PR should fix duplicates for successful and failed autowithdrawals.
Additional context
Can't reproduce bug locally because it only occurs in prod with multiple workers running.
Checklist
Are your changes backwards compatible? Please answer below:
Yes
Did you QA this? Could we deploy this straight to production? Please answer below:
Yes, tested successful autowithdrawal and failed autowithdrawal. Logs were still created fine.
For frontend changes: Tested on mobile? Please answer below:
Did you introduce any new environment variables? If so, call them out explicitly here:
Summary by CodeRabbit
New Features
Refactor