ordinals / ord

👁‍🗨 Rare and exotic sats
https://ordinals.com
Creative Commons Zero v1.0 Universal
3.85k stars 1.38k forks source link

Require bitcoin 24 for everything #1686

Closed casey closed 1 year ago

casey commented 1 year ago

I think this might make it simpler, not sure, open to discussion.

gnoblin commented 1 year ago

why would this make it simpler? thanks

raphjaph commented 1 year ago

makes codebase simpler

andrewtoth commented 1 year ago

Which config options or rpc commands introduced in 24 does ord make use of?

casey commented 1 year ago

Which config options or rpc commands introduced in 24 does ord make use of?

Good question!

rawtr descriptiors, for backing up the reveal transaction's pre-tweaked taproot key. (Pre-tweaked with the reveal transactions tapscript.)

friedger commented 1 year ago

This is bad for Bitcoin Knots users (currently, only bitcoin 23)

jtraub91 commented 10 months ago

This seems non-ideal, since it forces users to upgrade their node in order to use ord

raphjaph commented 10 months ago

Our wallet uses specific taproot and descriptor wallet features that only 24 and up support. You can still run the explorer with a lower version but no wallet commands.