symbol / mobile-wallet

Symbol Mobile Wallet
Apache License 2.0
8 stars 14 forks source link

Wallet backup got broken file #284

Closed postoronnii closed 2 years ago

postoronnii commented 2 years ago

WTR:

  1. Backup existing wallet
  2. Open .pdf file

Result: the file is broken

cryptoBeliever commented 2 years ago

Steps:

  1. Add at least one additional account
  2. Do PDF backup (Settings -> Backup)

Result: All pages (except first one where is correctly exported mnemonics) are presented incorrectly (without QR'codes, private keys and valid addresses)

I checked builds and looks like it was introduced in: https://github.com/symbol/mobile-wallet/releases/tag/1.1

I assume it could be "Added backward compatibility for previous XEM Opt-in accounts"

OlegMakarenko commented 2 years ago

Fixed in the: https://github.com/symbol/wallets-lib/pull/4 Awaiting test and review.

OlegMakarenko commented 2 years ago

Merged into main.