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.
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
CIP-0089? | Distributed Dapps & Beacon Tokens #466
base: master
Are you sure you want to change the base?
CIP-0089? | Distributed Dapps & Beacon Tokens #466
Changes from 19 commits
fa3f796
00057a9
beda2a3
f02e763
1976ab2
44c44e4
9b5e202
3b55d61
17a3f46
c0346aa
00ce8cb
ad9eda8
aa3f590
bd60aec
532b4f7
3bcd38f
341ac27
d78d315
daeeebf
c2877c8
fb24ba3
d3c643e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it's relevant that the huge community support for at least the non-custodial DEX applications that would be facilitated by Beacon Tokens should be accumulated in:
@AntMeyer1 at the CF also offered to help with this, but mainly it's your responsibility as authors to include these references as advocacy for your proposal. So far I've bookmarked (cc @KtorZ):
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Continuing this request in #418 (review) because it appears to relate to all use cases that might also avail of Beacon Tokens. TL;DR these issues seems so scattered across the DeFi field that someone might have to form a Working Group to clear out a path forward.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are good reference implementations but they're mentioned like anecdotes instead of a technically explicit process of how the Beacon Tokens are used in each... as CIPs normally have in their Specification section.
@SebastienGllmt mentioned this in the meeting so maybe he could explain more about how this could be done to achieve a more rigorously written proposal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beacon Tokens are highly generalizable, so we must find a balance in specifying how they should be used; without being overly vague or too specific. To that end, we figure it'd be worthwhile to wait until a few more reference implementations of distributed dApps that use beacon tokens are ready (currently being worked on), so that we can reference them in the CIP, and extract from them a general pattern of Beacon Token usage.
@fallen-icarus
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering that this may be the most commercially interesting & community supported use case, can you combine it with the 2 above for a list of 3 total? Do they need to be inventoried separately? If not, I think it would be more compelling & readable if all 3 applications were combined in a single section of potential or reference implementations & exactly how Beacon Tokens are used in each of them.