Matin Zadeh Dolatabad comments

Results 252 comments of


                                            Matin Zadeh Dolatabad

fix(analytics): typo in type declarations

Yeah actually it was my fault here :)) https://github.com/invertase/react-native-firebase/pull/7963 This got lost in between of all those changes.

[ENHANCEMENT] Remove async storage from the core implementation

A possible way/workaround for this is to import async storage in the try/catch block with a local variable in a helper class and export it to other parts of the...

feat(expo): upgrade Expo to v53 and React Native to 0.79

I am not sure why nx is using specific version of metro modules. Isn't it possible for nx to not rely on specific version of metro? @leggomuhgreggo When do you...

feat(expo): upgrade Expo to v53 and React Native to 0.79

@hassankhan Do you want to request a review again?

add three more react-native libs

Nice! Thanks @robertherber

Lottie-react-native (v7.0.0) Auto-Firing Events (e.g., onAnimationFinish) on Initialization in RN 0.78.2 + New Arch

Can you test the latest version and let me know?

Lottie crashes on android with react native 0.74.3

Hey! Can you try sharing the logcat output of the native crash with Android Studio?

[BUG] High async storage usage

> When it comes to Optimizely SDK, we have working `persistentCacheProvider` in place. You can implement your own cache provider with following interface - Yeah I have done that but...

`Duration must be set with physical times` error for aggregateGroupByPeriod on Anroid 13 and below

This might be related to https://stackoverflow.com/a/74468785/7978635. I might need to refactor the code to use `LocalDateTime` instead of `Instant`. https://developer.android.com/health-and-fitness/guides/health-connect/develop/aggregate-data#duration

`Duration must be set with physical times` error for aggregateGroupByPeriod on Anroid 13 and below

I can't promise until I find some free time. I am also working full-time and I am not paid to do work here.