Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Transition Overlay displayed when wallet was created in previous version #11506

Closed
LaurenWags opened this issue Oct 13, 2017 · 4 comments
Closed

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Oct 13, 2017

Description

Transition Overlay is displayed in 0.19.49 when wallet was created with 0.19.48.

Steps to Reproduce

  1. Clean install 0.18.36.
  2. Check for Updates > update to 0.19.48.
  3. Go to about:preferences#payments
  4. Enable payments. Wait for wallet to be created.
  5. Navigate to Advanced.
  6. Turn on update to preview releases, restart Brave.
  7. Check for updates > Update to 0.19.49.
  8. Navigate to Payments.
  9. Transition Overlay is displayed.

Actual result:
Transition overlay is displayed, but since my wallet was previously created this is confusing and seems unnecessary.
overlay

Expected result:
No transition overlay displayed.

Reproduces how often: [What percentage of the time does it reproduce?] 100%

Brave Version

about:brave info:
Brave | 0.19.49
rev | 0f0a5d6
Muon | 4.4.28

Reproducible on current live release:
no

Additional Information

found on MacOS, reproduced by @srirambv on Win.
Here is ledger_verbose output from Mac:
ledger_verbose_output copy.txt

Also, I was not able to reproduce this when starting with a clean install of 0.19.48 (instead of a clean install of 0.18.36).

@NejcZdovc
Copy link
Contributor

really nice steps, will take a look on monday

@LaurenWags
Copy link
Member Author

Still seeing issue using STR in description when updating to 0.19.52 on MacOS.

overlay-11056

@LaurenWags LaurenWags reopened this Oct 16, 2017
@NejcZdovc
Copy link
Contributor

With this fix we are only covering feature releases. For example scenario you upgrade from 0.18.36 to 0.19.52 (where this fix landed) to 0.19.53. In this scenario this error shouldn't happen anymore.

@LaurenWags
Copy link
Member Author

verified working on MacOS using the steps from #11506 (comment)

syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Resolves brave#11506

Auditors:

Test Plan:
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.