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

FATIP-102 Token Listing Standard #23

Open
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@drkatz
Copy link
Contributor

drkatz commented Nov 18, 2018

Rework spec and bring up to date with FATIP-100 & FATIP-101. Revise FATIP-101's standard index chain derivation.

@drkatz drkatz requested review from PaulBernier and AdamSLevy Nov 18, 2018


A single chain, the index chain holds all entries supporting the standard
The index chain holds entries corresponding to individual signed listings of FAT tokens by their issuers.

This comment has been minimized.

@PaulBernier

PaulBernier Nov 18, 2018

Contributor

You are using the term listing which I personally think is much better than indexing used elsewhere in this document. When I read indexing I thought it was a mechanism or structure allowing queries of tokens by multiple criteria such as date of creation, symbol etc. I believe naming it a listing or register spec fit better what this really is about.

This comment has been minimized.

@drkatz

drkatz Nov 19, 2018

Contributor

Where do you propose the term "listing" to be used instead of indexing?. The data on the chain is mainly intended to help a program create an index of tokens that can be searched like you are mentioning, but the data on chain itself is not an index, just references.

This comment has been minimized.

@PaulBernier

PaulBernier Nov 20, 2018

Contributor

I'd replace indexing/index pretty much everywhere from that doc. That document itself is called "Token Indexing Standard", so "Token Listing/Register Standard".
Indexing of tokens is one potential use case this standard, but it is not what this standard define in itself, it defines a way for tokens to register themselves publicly.

This comment has been minimized.

@PaulBernier

PaulBernier Nov 20, 2018

Contributor

I can make the actual change if you guys agree on the idea, no problem

This comment has been minimized.

@drkatz

drkatz Nov 20, 2018

Contributor

I think it's a better term, sorry I didn't mention that haha 🚀 Sure go ahead

This comment has been minimized.

@drkatz

drkatz Nov 24, 2018

Contributor

Thanks for the commits 🚀 Looking & reading better

Show resolved Hide resolved fatips/102.md Outdated
Show resolved Hide resolved fatips/102.md Outdated
Show resolved Hide resolved fatips/102.md Outdated

PaulBernier and others added some commits Nov 18, 2018

@PaulBernier PaulBernier changed the title FATIP-102 Token Indexing Standard FATIP-102 Token Listing Standard Nov 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment