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

Upgrade to Expo SDK 38 #170

Merged
merged 8 commits into from
Jul 4, 2020
Merged

Upgrade to Expo SDK 38 #170

merged 8 commits into from
Jul 4, 2020

Conversation

wangannie
Copy link
Member

What's new in this PR

  • Upgraded to Expo SDK 38 and updated packages accordingly
    • Changed Linking from react-native to expo-linking
    • Changed AsyncStorage to @react-native-community/async-storage
  • Refactored ProductsScreen, ProductDetailsScreen, and StoreDetailsScreen to functional components
  • Resolved [ERROR] ExpoHaptics requires VIBRATE permission for Android #167 by adding VIBRATE permissions to Android

How to review

n/a

Relevant Links

Online sources

https://blog.expo.io/expo-sdk-38-is-now-available-ab6cd30ca2ee
https://github.com/expo/expo/blob/master/CHANGELOG.md

https://stackoverflow.com/questions/61226530/typeerror-undefined-is-not-an-object-evaluating-reactnative-animated-text-pr

Related PRs

n/a

Next steps

Submit new standalone builds to the app stores

Screenshots

n/a

@wangannie wangannie merged commit 647e58a into master Jul 4, 2020
@wangannie wangannie deleted the @wangannie/upgrade-expo branch July 4, 2020 22:54
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.

[ERROR] ExpoHaptics requires VIBRATE permission for Android
1 participant