forked from ordinals/ord
-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore(api): update to 0.18.5 #10
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously, we were checking if an etching commit transaction had matured by asking bitcoind how many confirmations it had. This was incorrect, because if ord was behind bitcoind, the number of confirmations would be based on bitcoind's current block height, and would be greater than the number of confirmations the commit transaction had as of the reveal transaction block height. This commit fixes this by calculating the number of confirmations using the block height of the commit transaction, which is correct even if bitcoind is ahead.
This allows inscribing only the delegate without having to specify a file. In the batch it allows you to comletely omit the file as well.
Future runes features may be opt-in, for example, if they increase light client validation costs, or if they are simply too degenerate. This commit adds a "turbo" flag, which, if set, premptively opts in to such future features. It also sets the "turbo" flag on the genesis rune, UNCOMMON•GOODS. Because the genesis rune is inserted into the index when it is created, the schema version is also incremented, to force rebuilding indices created before this commit.
Document that data push opcodes 0 through 78 inclusive are allowed in runestones, and opcodes 79 and above will produce a cenotaph. Also add a test that makes sure that this is actually true.
Ord now requires Rust 1.74 to build, update `rust-version` in Cargo.toml files to reflect this.
Test that if the `cap` field of an etching is unset, it is treated as being zero, and the rune is unmintable.Test that if the `cap` field of an etching is unset, it is treated as being zero, and the rune is unmintable.
- Bump version: 0.17.1 → 0.18.0 - Update changelog - Update changelog contributor credits - Update dependencies
ordinals#3515) While waiting to broadcast a rune etching, take into account the fact that the commit transaction will have an additional confirmation in the next block. Previously we weren't, so the wallet was waiting for one confirmation too many.
- Bump version: 0.18.0 → 0.18.1 - Update changelog - Update changelog contributor credits
Don't allow etching if the etching reveal height is below the runes activation height, otherwise the user would waste funds on an ineffective etching.
…/<INSCRIPTION_ID> (ordinals#3209) In order for inscriptions to retrieve their own inscription ID, inscription content must be served from URLs with path `/content/<INSCRIPTION_ID>`. When an inscription with ID X delegates to another inscription with ID Y, meaning that X has a delegate field whose value is Y, when requesting the content of inscription ID X, the content of inscription Y must be returned from the URL with path `/content/X`. This allows inscription Y to use the ID of delegating inscriptions as seeds for generative content, since the delegating inscriptions will all have different inscription IDs.
`text/javascript` is the preferred MIME type for JavaScript, so use it instead of `application/x-javascript` when inscribing `.mjs` files.
Output `/output/<OUTPOINT>` was listed twice.
- Bump version: 0.18.3 → 0.18.4 - Update changelog - Update changelog contributor credits - Update dependencies
danadi7
force-pushed
the
ordzaar-master-0-18-5
branch
from
May 25, 2024 19:03
659cd0a
to
cd85071
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.