Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option to disable automatic payouts via config file #932

Closed
lukeskinner opened this issue Apr 29, 2019 · 2 comments
Closed

Option to disable automatic payouts via config file #932

lukeskinner opened this issue Apr 29, 2019 · 2 comments
Labels
improvement Improvements or enhancements to existing features

Comments

@lukeskinner
Copy link

lukeskinner commented Apr 29, 2019

Would you accept a PR that allows me to turn off the automatic ETH transaction payouts in the node?

The PR (at a glance) will probably be changing the /modules/command/dh/dh-pay-out-command.js file to check the config for a new setting in the execute function.

The reason for this being I'm currently paying outs offers for my nodes via the OT Hub website. Doing it this way I can do the payout via the management wallet and at a low gas price - this then has no effect on the node and it's operational wallet.

Example of the cost savings from lowering gas price by doing payouts externally:
The default 20 Gwei gas price on the operational wallet costs 0.00316291 ETH ($0.49) per payout on average.
Doing the payout at 2 Gwei gas price on the management wallet costs 0.00032378 ETH ($0.05) per payout on average.

As a side note OT Hub also checks all the conditions within the holding smart contract so it will block the payout transaction from being sent if it will fail anyway in the smart contract.

@kipliklotrika
Copy link

Hi @lukeskinner,

Everything you said makes sense and something like you proposed could be implemented. The actual change should be in modules/command/dh/dh-offer-finalized-command.js where payout command is issued. We'll get into it soon so no need for PR now.

@kipliklotrika kipliklotrika added the improvement Improvements or enhancements to existing features label Apr 30, 2019
@lukeskinner
Copy link
Author

Thanks for adding this feature! I'll enable it on my nodes later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvements or enhancements to existing features
Projects
None yet
Development

No branches or pull requests

2 participants