Skip to content
This repository was archived by the owner on Feb 23, 2021. It is now read-only.

Conversation

@tanx
Copy link
Contributor

@tanx tanx commented Nov 2, 2018

Closes #586

@tanx tanx requested a review from valentinewallace November 2, 2018 14:17
Copy link
Contributor

@valentinewallace valentinewallace left a comment

Choose a reason for hiding this comment

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

Sehr gut :D The components built in the set password PR integrated nicely!


const MainStack = createStackNavigator(
{
Password,
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you test these new views by commenting out the set password parts of the stack? I couldn't figure out any other way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can also tests the screen in the browser using regular storybook. But yes, I commented out the other screens to test the password view on mobile.

},
});

const PasswordView = ({ store, wallet }) => (
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good!

@tanx tanx merged commit 321a6f7 into master Nov 2, 2018
@tanx tanx deleted the password-mobile branch November 5, 2018 12:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants