-
Notifications
You must be signed in to change notification settings - Fork 166
Notification crashes yet. #293
Comments
Thanks for the report @blmhemu. What artifacts are you using in your Gradle buildscript? |
|
Hmm then that's weird; I don't recall seeing that. I'll take a look when I have some time. Perhaps we can incorporate your fix otherwise. |
|
any update over this issue? I am also experiencing the notification issue with targetsdk 29 and implementation 'org.ros.android_core:android_core_components:0.4.0' The notification dialog asking for shutting down crashes, leaving all ros nodes registered in the master. After a long time without working with my rosjava apps, I started updating all libraries (including android-studio, SDK, android UIs, rosjava dependencies, etc.) and eventually I noticed this problem with the shutdown. |
confirmed. The maven dependency is not updated. One could download the code for android_core lib, update the NodeMainExecutorService.java with the fix by @blmhemu (https://github.com/blmhemu-rosjava/android_core/blob/kinetic/android_10/src/org/ros/android/NodeMainExecutorService.java), modify if some resource is missing (e.g., R.drawable.shutdown), compile and finally use the generated .aar in the own android ROS project. |
Hey, although the android_core (notifications) has been updated to work with targetsdk 28, when I make an app, it still crashes. (My guess, it has not been updated in maven repo !!). The error is same as #283 . I am running Android oreo 8.1. The app (notification) responds fine when targetsdk is set to 25.
The text was updated successfully, but these errors were encountered: