search

pendo-io / pendo-mobile-sdk

Pendo captures product usage data, gathers user feedback, and lets you communicate in-app to onboard, educate, and guide users to value
https://www.pendo.io
Other
57 stars 2 forks source link

Pendo native plugin is not available on test #81

Closed trajano closed 2 months ago

trajano commented 9 months ago

Platform + Version e.g IOS 16, Android

SDK Version "rn-pendo-sdk": "^2.22.3",

Framework Expo using React Navigation,

Describe the bug This message appears on Jest logs whenever the rn-pendo-sdk is loaded

console.info
 [rn-pendo-sdk]  Pendo native plugin is not available. Ignoring Call. Please note that if you are using Expo, Expo Go is not supported by Pendo because Pendo SDK has a native plugin that is not part of the Expo Go app.

I am thinking that this message should only be generated when PendoSDK.setup is actually called and not as a side effect of module load.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Logs If applicable, please add crash logs OR pendo logs (can be turned on with PendoManager.shared().setDebugMode(true))

Sample Code Code snippet or Sample project

Additional context Add any other context about the problem here.

MikePendo commented 9 months ago

It means that the plugin can't find the native module. Can you make sure that the native module is there? (We dont support the Expo GO app). We have mentioned it in our docs under the limitation section:

Note that Expo Go is not supported by Pendo because Pendo SDK has a native plugin that is not part of the Expo Go app. Pendo can be used in development builds only. You can read more about development builds [Adding custom native code with development builds](https://docs.expo.dev/workflow/customizing/)

MikePendo commented 9 months ago

@trajano Please let us know if our previous comment resolve the issue

trajano commented 9 months ago

No @MikePendo Ideally it shouldn't find the native module because it is running in a Jest test. Sentry at least doesn't put noise on the logs even though it knows it is not working.

MikePendo commented 9 months ago

@trajano My apologizes we usually see this error in situations when people use GO app. So I have mistakenly associated your issue with that one

MikePendo commented 9 months ago

@trajano So unfortunately its under the info log and I dont see any suggestion that can allow us to silent it currently. Would it be possible for you to mock it?

MikePendo commented 9 months ago

@trajano We have opened and internal ticket on our side to provide an option to set loging level api. (Unfortunately I cant promise any deadlines when it will be resolved)

DavidFaermanPendo commented 5 months ago

closed by mistake

udilevin commented 2 months ago

@trajano our next release (3.3) will decrease the logging significantly (only calls to Pendo API will add a log)