-
Notifications
You must be signed in to change notification settings - Fork 30
Update to sentry-android 2.0.2 #41
base: master
Are you sure you want to change the base?
Conversation
@danielgek @bradmartin can somebody have a look at this? |
While waiting for the merge have a look at our fork at https://github.com/FinanzRitter/nativescript-sentry and the npm package https://www.npmjs.com/package/@finanzritter/nativescript-sentry |
Hey guys, i'm really sorry but i can't keep on maintaining this project, @sc85 are intrested in taking hover ? i can tranfer the npm package to you and put a deprecation note on this repo linking to yours |
or @bradmartin if you are also intrested |
Hi @danielgek, we are interested in taking over if @bradmartin is okay with it. |
Yep, works for me. I don't have time ATM. Would possibly still contribute in spare time as I use the plugin in some projects :) |
@sc85 con you go over slack ? will be there, just ping me ;) |
@sc85 did you get npm rights to this package and contributor to the repo? I'd like to update the plugin for NS7 and would like to work with you on the latest that you have working :) nevermind, I found your release notes now 👍 https://github.com/FinanzRitter/nativescript-sentry/blob/master/src/package.json |
The Android SDK was not up to date. I updated it to sentry-android 2.0.2 which came with some required changes. In addition I also updated the iOS SDK.
BREAKING CHANGES:
On Android Extras now get stringified before sent to the Sentry API. I could not find any other way.
The DSN needs to be added to the AndroidManifest (see README.md) since the I found no way to call the new native init method which expects a Lamda function