Skip to content

Conversation

@Oscar-Pepper
Copy link
Contributor

@Oscar-Pepper Oscar-Pepper commented Dec 3, 2025

ontop of #2031

fixes the missing checkpoint bugs causing errors with truncation during re-org and anchors during send.

tested by adding logs during testnet sync and checking the checkpoints are being created correctly, logs are included in previous commit (6d6696c) before cleanup for merge.

fixes #2012
fixes #2015
fixes #2017
fixes #1872

dorianvp
dorianvp previously approved these changes Dec 4, 2025
Copy link
Member

@dorianvp dorianvp left a comment

Choose a reason for hiding this comment

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

Nice!

@Oscar-Pepper Oscar-Pepper dismissed dorianvp’s stale review December 4, 2025 19:59

The merge-base changed after approval.

@Tomas-M
Copy link
Contributor

Tomas-M commented Dec 6, 2025

there are failing tests preventing this from merge,
looks like the test is downloading some zcash params which fails (connection reset by peer)
I am eager to try this fix asap, please update this PR to pass

@Oscar-Pepper Oscar-Pepper merged commit 2554abb into zingolabs:dev Dec 8, 2025
17 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.

Loading the ShardTree and truncating results in error Error sending transaction PANIC at pepper-sync/src/wallet/traits.rs Zero spendable balance

3 participants