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

Add advanced settings option to disable new address generation by default #910

Merged
merged 6 commits into from
Jul 5, 2024

Conversation

sneurlax
Copy link
Member

@sneurlax sneurlax commented Jul 3, 2024

The "Generate new address" button still exists and works.

@sneurlax
Copy link
Member Author

sneurlax commented Jul 3, 2024

Closes #832

@julian-CStack
Copy link
Collaborator

Needs more work. Currently setting appears in all wallets but is ignored for anything besides an electrumx based coin. Also checkReceivingAddressForTransactions should not be called at all if setting enabled

@sneurlax
Copy link
Member Author

sneurlax commented Jul 4, 2024

OK, checkReceivingAddressForTransactions isn't called if the pref is set

Except for on recovery, because at that point reuseAddress can't even have been set

My only lingering issue with this feature is that I want all addresses to be checked up to the gap limit, but only want the last manually-generated address to be shown. That would make this feature a little more practical

@julian-CStack julian-CStack merged commit bfd2845 into staging Jul 5, 2024
1 check failed
@julian-CStack julian-CStack deleted the reuseaddrs branch July 5, 2024 17:24
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