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

[Wallet] Some sapling wallet improvements #2887

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

panleone
Copy link

@panleone panleone commented Sep 15, 2023

first commit: filter sapling credit/debit only when the appropriate flag has been set in the filter
second commit: in the qt wallet don't generate shield addresses that have already been used

@panleone panleone self-assigned this Sep 15, 2023
@panleone panleone added this to the 6.0.0 milestone Sep 15, 2023
@panleone panleone changed the title [Wallet] Filter Sapling data only when the right flag is set [Wallet] Some sapling wallet improvements Sep 16, 2023
Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 74f7f94

Copy link
Member

@Liquid369 Liquid369 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK 74f7f94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants