You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed that in the discord chat, people were confused where they could get their public key, and what the export address was. In fact, it is a bit confusing for someone who isn't a developer.
There are two potential fixes. You can decide which one to pick:
We make both addresses consistent, and display a message
Ironfish wallet:export
Account: abc
Private key: xyz
Keep this address in a secure place. You can use it later to recover your wallet
Ironfish wallet:address
Account: abc
Public key: xyz
You can use this address for sending and receiving IRON.
Same as above, and we update the export name to privateKey, and address name to publicKey
Ironfish wallet:privateKey
Private key: xyz
Keep this address in a secure place. You may use it later to import your wallet
Ironfish wallet:publicKey
Public key: xyz
You may use this address for sending and receiving IRON.
This is a quick fix, so I can get started on fixing it.
The text was updated successfully, but these errors were encountered:
Yes, I agree. But adding in the message after displaying the private, and public key would be useful for current and future users, which I suggest in the feature request. Currently we don't have any message displaying, which I feel is important when displaying the keys to our users.
We can also change the name of export, and address to privateKey, and publicKey, but this would be up to the developers, whether they might prefer it that way or not.
I have updated the description, with the before output. This should make it a lot easier to understand
Description
I noticed that in the discord chat, people were confused where they could get their public key, and what the export address was. In fact, it is a bit confusing for someone who isn't a developer.
Currently we have:
There are two potential fixes. You can decide which one to pick:
This is a quick fix, so I can get started on fixing it.
The text was updated successfully, but these errors were encountered: