Conversation
|
We have the same prefix for Equilibrium and Genshiro networks |
|
We need to be able to connect Ledger devices to both networks. |
|
What do you mean by connect Ledger device? You will need to have a Ledger App oder connect through the Polkadot extension to Ledger, no? |
Yep :) https://github.com/polkadot-js/common/blob/master/packages/networks/src/substrate.ts#L9-L19 |
Currently our network are using 11 assets and its count will be increased. What is the best solution about symbols naming? Use just one main token name? |
I would say that the idea was to have all of them in the document, because of this reason the tokens are a list. |
Well, then it will look like this:
The first three assets are created by ourselves, others are bridged from other networks. Do I need to set only ours or all assets in the 'symbols' field? |
|
I am more concerned about the requirement for the uniqueness of the prefix for networks. We already have our second project Equilibrium. By using the Polkadot js extension we can already connect ledger devices to Equilibrium, but with this limitation, we can't add Genshiro, because both networks have the same prefix |
Wdyt @jacogr ? |
IMO This should only set the native tokens of the network. Other assets should come with some asset metadata, either on-chain or in your own Equilibrium/Genshiro registry. |
|
Ok, I updated it. |
ss58-registry.json
Outdated
| "network": "genshiro", | ||
| "displayName": "Genshiro Network", | ||
| "symbols": ["GENS", "EQD", "LPT0"], | ||
| "decimals": [9], |
There was a problem hiding this comment.
Should be the same length as symbols.
Is there a reason that the two networks have the same prefix? The prefixes should make addresses identifiable as belonging to separate networks. |
Because the same address is more simple to use for our users. We have two projects with started mainnets. |
|
It's going to be tough to resolve. When I wrote the spec for this registry, I decided that the network name should be unique to a prefix. Changing that could have downstream effects on tools that might use network names to identify entries in this registry. Changing |
|
Ok, here is our solution: we changed the Equilibrium prefix to resolve this problem. Please, review it ASAP |
|
LGTM but @BenWhiteJam manages the approvals of these |
ss58-registry.json
Outdated
| "prefix": 68, | ||
| "network": "equilibrium", | ||
| "displayName": "Equilibrium Network", | ||
| "symbols": ["Unknown", "USD", "EQ", "ETH", "BTC", "EOS", "DOT", "CRV"], |
There was a problem hiding this comment.
@joepetrowski Following your comments, wouldn't it be also recommended to list only chain native tokens for the Equilibrium network then?
@BenWhiteJam done |
No description provided.