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

Change Address from Sparrow No Longer Shown #193

Open
shannona opened this issue Jul 12, 2023 · 2 comments
Open

Change Address from Sparrow No Longer Shown #193

shannona opened this issue Jul 12, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request ux User experience or interface

Comments

@shannona
Copy link
Contributor

When Sparrow generates a transaction with change, GST no longer recognizes the money sent back to the same key as change. (It used to properly differentiate it.) It's just an additional "sent" address instead.

IMG_6172
IMG_6173

@shannona shannona added bug Something isn't working ux User experience or interface labels Jul 13, 2023
@wolfmcnally
Copy link
Collaborator

Unfortunately, when we updated to a newer version of LibWally in this last iteration, which has better segwit support, the new version invalidated how change outputs were determined. The code we were using for this was originally inherited from an open source package. This code has been disabled, pending prioritization of figuring out the right way to do it now. So I'm going to change this to "enhancement," and we should prioritize it, as it may take some time to research, implement, and test. In the meantime, it would be useful to post a test vector here.

@wolfmcnally wolfmcnally added the enhancement New feature or request label Jul 15, 2023
@shannona
Copy link
Contributor Author

I've included a video of Sparrow's Animated QR as a Test Vector of a PSBT that should break out a change address:

IMG_6213.mov

I've also removed the bug tag.

@shannona shannona removed the bug Something isn't working label Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux User experience or interface
Projects
None yet
Development

No branches or pull requests

2 participants