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

Update Part-10 instructions, links, library version #3862

Open
wants to merge 8 commits into
base: source
Choose a base branch
from

Conversation

EledenGreen
Copy link
Contributor

  1. update instruction regarding installation of Expo Go app in part10a.md
    • latest Expo Go in app store are for SDK 51 and are not compatible with the material.
    • added instruction to download for SDK 50.
  2. updated instruction in Ex: 10.9
    • TextInput of react-native doesn't have error prop
    • TextInput of react-native-paper have error prop
  3. update import statement in part10c.md
    • useAuthStorage.js and useSignIn.js are in the same directory: src/hooks
  4. updated sentence in part10c.md
    • "SecureStore is similar to AsyncStorage as a form of persisted storage, but it encrypts the stored data."
  5. fix typo in part10d.md
    • - "a quite"
    • + "quite a"
  6. fix: broken link in part10d.md
  7. update jest-expo install instruction in part10d.md
    • latest jest-expo version is v51 and is not compatible with the material
    • updated to install jest-expo@^50.0.0

for the error prop to work, the TextInput component must be of react-native-paper.
TextInput of react-native doesn't have error prop
useAuthStorage.js and useSignIn.js are in the same directory: src/hooks
update sentence to: 
"SecureStore is similar to AsyncStorage as a form of persisted storage, but it encrypts the stored data."
- "a quite" 
+ "quite a"
latest jest-expo is v51 and is not compatible with expo SDK 50
@vejol vejol added the part 10 label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants