Kris Nuttycombe

Results 229 comments of Kris Nuttycombe

Between #1402, #1473 and #1496 we now believe this to be resolved.

We now allow having UFVKs corresponding to multiple distinct seeds to be stored in the wallet, so raising an error no longer makes sense.

I just successfully performed 1-conf shielding with Zashi 1.2.2 (which updates to the latest SDK release). So I'm closing this as fixed.

@aphelionz there's an underlying clippy CI issue being fixed in https://github.com/zcash/librustzcash/pull/1785 - you might want to rebase once that merges to clear this issue.

https://github.com/zcash/librustzcash/pull/1879 now builds atop this branch, merging `main` and addressing comments from review.

Converting back to draft because it's necessary to fix #5720 as a prerequisite to fixing `gettransaction` to only return legacy transaction information (`gettransaction` was also affected by the issue that...

> For context, the address was taken from the outgoing sapling change note, decrypted with the OVK. The incoming sapling change note was also successfully decrypted with the internally scoped...

Okay, yep, that looks like a zcashd bug. I can see how it's coming from the very weird implementation of https://github.com/zcash/zcash/blob/master/src/rpc/misc.cpp#L826-L852

Let me consult with @daira and/or @str4d on what we need to do here.