Skip to content

Conversation

@Davidson-Souza
Copy link
Member

What is the purpose of this pull request?

  • Bug fix
  • Documentation update
  • New feature
  • Test
  • Other:

Which crates are being modified?

  • floresta-chain
  • floresta-cli
  • floresta-common
  • floresta-compact-filters
  • floresta-electrum
  • floresta-watch-only
  • floresta-wire
  • floresta
  • florestad
  • Other:

Description

Closes #156

This commit implements BIP-0094-related pow changes, and makes sure that all node functionalities works with it the "testnet4" network.

Note to reviwers

I've managed to headers-sync, but didn't download blocks due to no bridges so far. I'll try to get one running

@Davidson-Souza Davidson-Souza added enhancement New feature or request consensus This changes something inside our consensus implementation labels May 22, 2025
@Davidson-Souza
Copy link
Member Author

Updated with @JoseSK999's comments

@JoseSK999
Copy link
Contributor

Was this able to sync with the bridge node?

@luisschwab
Copy link
Contributor

I guess there is no bridge yet, I'll try to get one running over the weekend.

@Davidson-Souza
Copy link
Member Author

@JoseSK999

Was this able to sync with the bridge node?

It headers-synced. But I'll try with a proper bridge node here and report back.

@JoseSK999
Copy link
Contributor

Oh actually headers sync is now testing the full change behavior

@Davidson-Souza
Copy link
Member Author

I've just remembered, utreexod doesn't do testnet4 yet. So headers sync is the best we can do rn 😂

Copy link
Contributor

@JoseSK999 JoseSK999 left a comment

Choose a reason for hiding this comment

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

Ok so some final comments

This commit implements testnet4-related pow changes, and makes sure that
all node functionalities works with it.
@Davidson-Souza
Copy link
Member Author

rebased after #506 and updated with @JoseSK999's comments.

Copy link
Contributor

@JoseSK999 JoseSK999 left a comment

Choose a reason for hiding this comment

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

ACK 8546ba3

@Davidson-Souza Davidson-Souza merged commit 247c608 into vinteumorg:master Jun 2, 2025
10 checks passed
@luisschwab
Copy link
Contributor

@Davidson-Souza does your bridge work with testnet4?

@luisschwab
Copy link
Contributor

Answer: not really.

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

Labels

consensus This changes something inside our consensus implementation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

testnet4 support

3 participants