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

Sync integration send #1649

Merged
merged 10 commits into from
Feb 11, 2025
Merged

Conversation

Oscar-Pepper
Copy link
Contributor

@Oscar-Pepper Oscar-Pepper commented Feb 6, 2025

there will be a second part to this PR that enables the completion and broadcast of txs

this PR:

  • enables propose and re-integrates a lot of related functions and types
  • implements zcash client backend (ZCB) traits on the wallet. some impls copied from TxMap and TransactionRecordsById and some are re-implemented
  • note selection has been reworked to use the new clean wallet interfaces
  • many methods have been upgraded or reworked to make use of the new access to ALL wallet data. this was previously impossible due to RWlocks on parts of the wallet.

@Oscar-Pepper Oscar-Pepper marked this pull request as ready for review February 10, 2025 09:01
@Oscar-Pepper Oscar-Pepper marked this pull request as draft February 10, 2025 09:03
@Oscar-Pepper Oscar-Pepper marked this pull request as ready for review February 11, 2025 03:43
@dorianvp dorianvp merged commit 9571836 into zingolabs:dev_2-0 Feb 11, 2025
15 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants