search

Concordium / concordium-reference-wallet-ios

Reference wallet for the Concordium blockchain
Apache License 2.0
11 stars 10 forks source link

Import/export across *nets #11

Closed concordium-cl closed 3 years ago

concordium-cl commented 3 years ago

Description Make sure that import/export accross nets (e.g. import a stagenet wallet into a testnet app or a testnet wallet into the mainnet app and vice versa) fails with an understandable, informative error message to the user.

Steps to Reproduce export net --> import net stagenet --> testnet testnet --> stagenetnet stagenet --> mainnet mainnet --> stagenetnet testnet --> mainnet mainnet --> testnet

Expected Result Failure but with informative error message to user.

Actual Result Unclear.

concordium-cl commented 3 years ago

@jens-concordium: Could you provide text for error message please?

jens-concordium commented 3 years ago

Error message suggestion: "Files exported on Concordium Mainnet, TestNet or StageNet are not compatible with the other chains. Please import on the correct chain."

jens-concordium commented 3 years ago

I tried this in StagingNet v0.1.74 and Concordium ID (Testnet) v0.1.67, and I see no noticeable difference. It still gives the same, somewhat unclear, error.

iOS Import failed

jens-concordium commented 3 years ago

Tested again on Staging v 0.1.75 and TestNet v 0.1.68, and it now seems to work as intended.

jens-concordium commented 3 years ago

Now also tested on Mainnet app version 1.0.2, on which it also works as intended.