-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Show stable inscription sequence number when listing inscriptions (both wallet and website) #1455
Comments
One concern with surfacing the inscription number right now is that inscription numbers depend on knowing all inscriptions before a given inscription, and when we update the protocol, there will be new inscriptions which old clients ignore, which will give different clients a different view of inscription numbers. So for example, we haven't made a endpoint that lets you look up inscriptions by number, because those URLs might be unstable. So given that numbers are unstable, should we surface them in the CLI? |
As just discussed - having a stable sequence numbering scheme would be high value and should be doable. We need to make sure to also count inscriptions that are invalid from a perspective of the current client version to make this stable across future upgrades. We should get the multiple inscriptions per transaction changes implemented before working on this issue. |
Executive decision by @casey to keep existing numbers stable: https://twitter.com/rodarmor/status/1626621385618628608 Details to be worked out. |
I think this ship has sailed in the sense that people are attached to the existing inscription numbers, so in true Bitcoin fashion, we now how to work around our past mistakes. Context:
We are now committed to the existing numbers. New priorities:
|
Where can I find details about the bug? |
Is this bug related to https://github.com/casey/ord/issues/1841 or is it a different issue? 🤔 |
Show the number of the inscription as shown on the website when listing inscriptions from the wallet.
Planning to work on this.
The text was updated successfully, but these errors were encountered: