React Native 0.85#8
Conversation
JP (jpdriver)
force-pushed
the
jpdriver/main
branch
from
7567b8e to
bdb1521
Compare
Stephanie Cure Chegwin (Scure21)
requested a review
from Luis Camargo (LuisCamargoM)
| Use the localStorage polyfill for key-value storage. **Never use AsyncStorage** | ||
|
|
||
| ```tsx | ||
| import "expo-sqlite/localStorage/install"; |
There was a problem hiding this comment.
Why is it preferred to use the polyfill Vs expo-sqlite/kv-store for key-value storage?
There was a problem hiding this comment.
was introduced in this PR
at a guess maybe the intent is to have the same API across both native and web (where AsyncStorage isn't available)
but also to be clear, i'm not checking or re-wording anything in the .agents/skills -- these are just pulled in from expo/skills completely as-written
There was a problem hiding this comment.
Ah! yeah makes sense! I always forget about web support LOL.
Stephanie Cure Chegwin (Scure21)
left a comment
Stephanie Cure Chegwin (Scure21)
left a comment
There was a problem hiding this comment.
Wow JP this is amazing! so many good guidances in the skills and the docs/adr is fantastic! Thank you for doing this!
I'm still reviewing but will finish today.
2 participants
(wasn't able to push this directly to
maindue to the TruffleHog restriction)see https://github.com/jpdriver/react-native-template/releases/tag/rn-0.85 for Release Notes
✌🏻