search

microsoft / clarity-apps

Repo for distributing Clarity Apps packages
MIT License
46 stars 2 forks source link

Package is still raw. Some areas to take a closer look: #2

Closed y-lukashevskyi closed 1 year ago

y-lukashevskyi commented 1 year ago

I've noticed that this package is only 2 months old, so I expected some inconsistency. Here are some things that this package does/breaks:

  1. It only stated in the src code that react-native-clarity doesn't support IOS. I think this must be added to README
  2. Linking with expo is broken. It cannot auto-link. Running pod install does nothing. Issue still persists
  3. On Android it messes up the appearance of the inputs. It looks like it overrides/cancels paddingHorizontal property.

Versions: react-native-clarity: 0.0.2 react: 18.1.0 react-native: 0.70.5 expo: 47.0.12

nouranmaher86 commented 1 year ago

Hello- Thank you for your comment. To answer #1 and #1, Clarity Mobile SDK only supports native Android or the Android parts of the following cross-platform platforms (React native, Ionic, Cordova). iOS is not supported yet. For #3, We have troubleshooted but were not able to replicate. Can you let me know your Clarity project ID and the app name that you have resulted in this bug?

ibradwan commented 1 year ago

For #3 we were able to reproduce it and worked on a fix that should be released soon enough, please stay tuned.

ibradwan commented 1 year ago

The fix has been released in the latest version (0.0.4), please give it another shot :)