Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0 6 1 fixes #419

Merged
merged 4 commits into from
Nov 7, 2018
Merged

0 6 1 fixes #419

merged 4 commits into from
Nov 7, 2018

Conversation

dafuga
Copy link
Contributor

@dafuga dafuga commented Nov 7, 2018

@dafuga dafuga requested a review from aaroncox November 7, 2018 08:01
@@ -58,7 +58,7 @@ const paneMapping = [
},
{
element: ToolsWallets,
modes: ['cold', 'hot', 'ledger', 'watch'],
modes: ['cold', 'hot', 'ledger', 'watch', 'skip'],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aaroncox could it be that you had skipped the setup the one time that the "Manage Wallets" section was hidden for you? Maybe that should show up in the sidebar even when the setup was skipped?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be! We can def test to see if this resolves it

@aaroncox aaroncox merged commit 78d640f into greymass:develop Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants