(Fix) Link to POA explorer for POA networks #78
Conversation
ghost
assigned 
old-ui/app/components/token-cell.js
Outdated
| const prefix = prefixForNetwork(network) | ||
| return `https://${prefix}etherscan.io/token/${tokenAddress}?a=${address}` | ||
| const POAnetwokIDs = [77, 99] | ||
| if (POAnetwokIDs.includes(parseInt(network))) { |
There was a problem hiding this comment.
Given that this is done in a lot of places in the code, maybe it's time to move it to some util function?
|
Related to @fvictorio comment, maybe it would be a good idea to extract the logic of getting the correct url for eth and poa networks into a npm package similar to https://github.com/MetaMask/etherscan-link since it's something it's repeated in several places of nifty wallet. This npm package could also provide the information related to the network (such as the name) based on the network id which is something repeat in several places too. (Example https://github.com/nya1/eth-net-type) This would be useful for other DApps too, for example Bridge UI which has the same logic implemented. |
|
@fvictorio @patitonar thank you for valuable comments! |
Relates to #77
Links to token's smart-contract to community POA explorers (Sokol/Core) are added when the current network is POA