Page MenuHomePhabricator
Differential D17196

[bip21] Introduce a spec for supporting simple token send txs
ClosedPublic
Actions

Authored by bytesofman on Sun, Nov 24, 15:07.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC0ca709530cf1: [bip21] Introduce a spec for supporting simple token send txs
Summary

This would make it easy for users to share token payment requests via QR code or cashtab url

While this is already possible to do using existing bip21 op_return_raw, it is not straightforward for non-developers to create such a request.

Test Plan

Proofread. To be implemented in Cashtab.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bytesofman created this revision.Sun, Nov 24, 15:07
Herald added a reviewer: Restricted Project. · View Herald TranscriptSun, Nov 24, 15:07
Harbormaster completed remote builds in B31300: Diff 51078.Sun, Nov 24, 15:07
bytesofman published this revision for review.Sun, Nov 24, 15:07
emack added a subscriber: emack.Mon, Nov 25, 01:28

is the exclusion of non-send token txs on the basis that they're not a use case for QR payments?

bytesofman added a comment.Mon, Nov 25, 02:20
In D17196#390403, @emack wrote:

is the exclusion of non-send token txs on the basis that they're not a use case for QR payments?

I think they could be included, but it would be more complicated. They need more params, potentially special handling depending on token type, etc.

So, for now, it's just that "send" is the first obvious use case that is being asked for. If we get a demand for other actions, we can add it in a way that doesn't interfere with this.

It makes sense for "send" to be the default token action...it's the default XEC action.

emack accepted this revision.Mon, Nov 25, 03:57
This revision is now accepted and ready to land.Mon, Nov 25, 03:57
Closed by commit rABC0ca709530cf1: [bip21] Introduce a spec for supporting simple token send txs (authored by bytesofman). · Explain WhyMon, Nov 25, 07:09
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABC0ca709530cf1: [bip21] Introduce a spec for supporting simple token send txs.

Revision Contents
Changeset List

PathSize
doc/
standards/
23 lines

Diff 51090

View Options

doc/standards/bip21-ecash-additions.md

Loading...