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

New Registration Screen #53

Merged
merged 28 commits into from
Sep 22, 2017
Merged

New Registration Screen #53

merged 28 commits into from
Sep 22, 2017

Conversation

edencakir
Copy link
Contributor

@edencakir edencakir commented Sep 9, 2017

About improving scroll animation performance; if these get merged I guess these animations will be way better. Until then we can also consider using gifs or a video:
facebook/react-native#15609
facebook/react-native#15610

Android inverted flatlist issue: facebook/react-native#14560

simulator screen shot 14 sep 2017 5 11 08 pm

@edencakir edencakir changed the title New Registration Screen WIP: New Registration Screen Sep 9, 2017
@edencakir edencakir changed the title WIP: New Registration Screen New Registration Screen Sep 14, 2017
@edencakir
Copy link
Contributor Author

@BilalBudhani @joshmadewell it'd be great if you guys can review this.

import { View, Text, TouchableOpacity, ScrollView, Dimensions } from 'react-native';
import { intro1, intro2, intro3, intro4 } from 'kitsu/assets/img/onboarding/';
>>>>>>> ac7ac011c40b7d0948eef59ca59ff6b91fec105f
Copy link
Contributor

Choose a reason for hiding this comment

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

@edencakir you missed resolving a conflict here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah thanks for the heads up. I'm not sure how I missed that.

@BilalBudhani
Copy link
Contributor

@edencakir when I try to sign in with my username & password; nothing happens after pressing sign in button, I just see loading indicator on the button forever. Please look into this.

@edencakir
Copy link
Contributor Author

@BilalBudhani This PR has nothing to do with Login or Signup screens (except facebook login). This is just a landing screen after onboarding. So for that, we should better look at the login screen.

@BilalBudhani
Copy link
Contributor

@edencakir Okay. I think #66 branch by @joshmadewell fixed that issue.

@edencakir
Copy link
Contributor Author

@BilalBudhani Great, let me go through this PR before merging.

@edencakir
Copy link
Contributor Author

@BilalBudhani Done, should be ready to merge

@BilalBudhani BilalBudhani merged commit b4f48af into hummingbird-me:master Sep 22, 2017
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