Skip to content

Add ability to configure coinbase verification in UI#210

Open
pavlenex wants to merge 1 commit into
stratum-mining:mainfrom
pavlenex:coinbase-config
Open

Add ability to configure coinbase verification in UI#210
pavlenex wants to merge 1 commit into
stratum-mining:mainfrom
pavlenex:coinbase-config

Conversation

@pavlenex

Copy link
Copy Markdown
Contributor

This PR adds a Coinbase Verification toggle to Advanced Options for solo pool mining in both setup and Settings. It is enabled by default and controls verify_payout, allowing miners to turn off the check that their payout address appears in the pool’s coinbase transaction.

Before

Always turned on by default
Screenshot 2026-07-13 at 13 57 55

After

Enabled by default, optional to disable

Screenshot 2026-07-13 at 14 06 23 Screenshot 2026-07-13 at 14 05 27 Screenshot 2026-07-13 at 14 07 00

@GitGab19

Copy link
Copy Markdown
Member

I think we should keep verify_payout optional here.

The application layer already defaults verify_payout to false when the field is not present in the config, and this option is only useful for SOLO scenarios. Making it mandatory in TranslatorConfig and normalizing it to true for every translator config makes it look like a general setting, even though it has no meaning for pool/JD modes.

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