-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Allow creating offers for runes #4183
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
Comments
Should this issue include sub-balance rune buy offers? It would be fairy easy to add to PR #4282 but I don't want to unnecessarily expand the scope if that should be a separate PR. |
By sub-balance rune buy offer, you mean an offer to buy less of rune X than the total amount in |
Gah, didn't mean to close T_T |
Sounds good! Should I also remove the functionality to handle multiple UTXOs and make that a separate PR? I added that this morning after re-reading the issue and seeing you mention UTXOs plural. In the interest of keeping PRs small, it could make sense to separate that out. |
Yah, I would say that should go in a follow up. |
Sounds good, I'll remove it from the PR. |
I actually think that, if it makes the PR simpler, we could also remove anything which requires adding a runestone. So, you can buy all the runes of a single type in a UTXO, and if there's anything else in there, it barfs. I just looked at the PR, and it's still very large, 3200, so I'm just thinking anything that can be removed should be. |
I'm also not sure if the commands should operate in different "buying runes" vs "buying inscriptions" modes. It really depends on which one makes the code simpler. |
We can do that. Looking like it'll be ~4 PRs: one for no-runestone offers, one for multiple runes in a UTXO, one for multiple UTXOs, and one for offers less than the balance held in a UTXO (not necessarily in that order).
We can talk about it but I personally do think it makes the code simpler separating it out. |
Sounds good, just posted on the PR, so you can follow up on my comments there. |
Offers for runes are trickier than for inscriptions, since the bidder must specify not only which rune they wish to buy, but which UTXO or UTXOs containing those runes they want to create an offer for.
As a first pass on this feature, we should allow:
ord wallet offer create --rune <AMOUNT>:<NAME> --utxo <UTXO>
ord wallet offer accept --rune <AMOUNT>:<NAME>
Where
UTXO
is the utxo containing the runes to bid on, is the amount to purchase, and<NAME>
is the name of the rune.The offer should send all other runes to the seller recipient address.
The text was updated successfully, but these errors were encountered: