-
Notifications
You must be signed in to change notification settings - Fork 112
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
Added functionality #13
Comments
Nice idea! I am all for making this list more useful. PR welcome, otherwise I can try to get to this sometime soon. |
Maybe it should be a /v2 document in case others are using it as it? Happy to issue a PR if you can devise a way for it to be backwards compatible. |
I think maybe we can generate both lists, one with more detail, and the one we are generating currently, and the users can pull in the list they want. |
Hiya boyz. Joining the pack here. and great work so far. I can see many usages of the uri data, and even on top of that, consider 'protocol' (POS, POW etc), 'Algorithm' if mined (POW). |
@angelexevior thanks for the kudos! :) How do you see the current list co-existing with this list that was proposed by @wmbutler above? The current list has a structure of:
whereas a new structure was proposed of:
As far at the API goes, I also see this being a problem if we try to keep backwards-compatibility (I wouldn't want to set a custom getter for each cc): cryptocurrencies.BTC;
//=> 'Bitcoin' So I am not seeing a way to make these two lists compatible, unless I am missing something? I think having two lists generated, and the user can decide which one they want to use is fine by me, but I am happy to consolidate the lists if there is a way to maintain backwards-compatibility. With all of this said, each time we do a release we bump the major version because the list changes pretty drastically each time we build it (and I can see use-cases where some apps are relying on old versions of currency names or symbols), so in a way we don't have to be so careful, we could change the API in a new major version. |
Apologies @radiovisual. Spoke before switching my brain into gear. Well, if we want to go forward with this, the only way is to introduce a separate V2.0 and announce that the original json will not be maintained anymore, thus keeping the existing integrations alive until they switch to the newer version. What say you! |
@angelexevior, another option is just maintain a few different lists, and the user just pulls in the list they need. There are use cases for having different types of lists (such as #16), so I would not be against having different lists, but then again, its more work. Perhaps there is some value in keeping one list, and then let the user generate their own ideal list if they need something a bit more stripped down. Either way is fine with me. |
Two lists is probably not a great idea. What about this?
|
@wmbutler I like your proposed structure {
"bts" : {
"name" : "Bitcoin",
"uri" : {
"coin": "bitcoin",
"amount" : "amount",
"message" : "message",
"label" : "label"
}
}
} 👍 |
D'accord @wmbutler @radiovisual Also, consider adding a small API framework to it (REST) for a wider reach. |
This list is generated semi-automatically by running
This is one of the utilities of a flat list like this, where anyone that pulls this list into their project can use it to generate anything they might need. Feel free to build/host anything you like that uses this list (help is always welcome) in a separate repo so we can still keep this flat list a small dependency. I would love to see some tooling spring up around this list. |
This list could be much more useful if it contained an object for each coin instead of a parameter list:
The URI object would be particularly useful because it's required for wallets that present QR codes. These URI's don't use the symbol, they use a long name for the currency an those lists aren't easy to find freely. Also, many coin URI's have a different set of terminology for the message. Some call it a memo. So, if you could normalize this through this library, it would be pretty powerful as well.
The text was updated successfully, but these errors were encountered: