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

Transition is invalid in current state #641

Closed
JimmyMow opened this issue Aug 1, 2018 · 2 comments · Fixed by #673
Closed

Transition is invalid in current state #641

JimmyMow opened this issue Aug 1, 2018 · 2 comments · Fixed by #673
Assignees
@mrfelton
Labels
status: needs review issue needs code review and testing type: bug 🐛 Something isn't working
Milestone
v0.2.2-beta

Comments

@JimmyMow
Copy link
Member

JimmyMow commented Aug 1, 2018 •
edited
Loading

When refreshing (cmd + r) in dev mode I get the following error

screen shot 2018-08-01 at 1 16 45 pm
@JimmyMow JimmyMow added the type: bug 🐛 Something isn't working label Aug 1, 2018
@mrfelton
Copy link
Member

mrfelton commented Aug 2, 2018

@JimmyMow out of interest, what is the use case of using cmd+r to refresh the page?

@mrfelton mrfelton added this to the v0.2.2-beta milestone Aug 11, 2018
@JimmyMow
Copy link
Member Author

@mrfelton for development. refreshing the page to get new state/new saved changes

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue Aug 15, 2018
@mrfelton
feat(wallet): restart onboarding on app refresh
489f115 
If a full refresh is done from within the app (`cmd + r`) restart the
onboarding process from the beginning, including stopping lnd and
disconnecting any active connections.

Fix LN-Zap#641
@mrfelton mrfelton mentioned this issue Aug 15, 2018
Merged
@mrfelton mrfelton added the status: needs review issue needs code review and testing label Aug 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mrfelton mrfelton

Labels
status: needs review issue needs code review and testing type: bug 🐛 Something isn't working
Projects
None yet
Milestone
v0.2.2-beta
Development

Successfully merging a pull request may close this issue.

feat(wallet): restart onboarding on app refresh mrfelton/zap-desktop
2 participants
@mrfelton @JimmyMow