Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

fix: back button after wallet reload #1785

Merged
merged 3 commits into from
Mar 9, 2020

Conversation

brenopolanski
Copy link
Contributor

Summary

Steps to recreate:

  • Launch Desktop Wallet
  • Select a wallet address
  • Reload the wallet (CTRL+R)
  • Select 'Back'
  • Nothing happens

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@ghost ghost added the Complexity: Low Less than 64 lines changed. label Mar 6, 2020
@codecov
Copy link

codecov bot commented Mar 6, 2020

Codecov Report

Merging #1785 into develop will decrease coverage by 0.02%.
The diff coverage is 11.11%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1785      +/-   ##
===========================================
- Coverage    59.55%   59.52%   -0.03%     
===========================================
  Files          367      367              
  Lines         9099     9103       +4     
  Branches      1860     1861       +1     
===========================================
  Hits          5419     5419              
- Misses        3494     3498       +4     
  Partials       186      186
Impacted Files Coverage Δ
.../components/Wallet/WalletDetails/WalletDetails.vue 6.95% <11.11%> (-0.26%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee97098...9c0d959. Read the comment docs.

@faustbrian faustbrian changed the title bug: fix back button after wallet reload fix: back button after wallet reload Mar 9, 2020
@alexbarnsley alexbarnsley merged commit 6cf66dc into develop Mar 9, 2020
@ghost ghost deleted the fix/wallet-details-back-button branch March 9, 2020 14:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Complexity: Low Less than 64 lines changed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants