#369: Implemented App Usage observation#383
Conversation
Other changes: - Better observation initalization process for less memory usage - Added a long running observation storage possibility, so data are not sent right away
There was a problem hiding this comment.
just some really minor things and a question - but code LGTM -> and your pipeline does need to be fixed
one open question: is it possible to take the tracking approvement away for one observation or is it similar to how we do with the study consent? can they take the approvement away through the app?
i still need to test the application on my phone
| navController: NavController | ||
| viewModel: ObservationDetailsViewModel | ||
| ) { | ||
| val backStackEntry = remember { navController.currentBackStackEntry } |
There was a problem hiding this comment.
or is that backStackEntry in the new Wrapper included?
There was a problem hiding this comment.
That was necessary before the wrapper. This is now handled by the wrapper
|
Tracking consent needs to be able to be declined at any moment, as per Apples rules. But this is the same as any other observation, we cannot do anything if the user decides to remove any sensor permissions for our app, we can only show a warning, but we cannot deny the usage |
# Conflicts: # shared/src/commonMain/moko-resources/base/strings.xml # shared/src/commonMain/moko-resources/de/strings.xml
|
.gradlw build didn't work when i checked the branch out for testing purporses |
…369-app-usage-observation
# Conflicts: # androidApp/src/main/java/io/redlink/more/app/android/activities/dashboard/filter/DashboardFilterView.kt # androidApp/src/main/java/io/redlink/more/app/android/activities/notification/filter/NotificationFilterView.kt
|
- GPS recording on iOS and Android - Permission request now fully works - Notification deeplinks no longer infinitley route to the notificationview when a notification is clicked without deeplink - Translation Strings added - Cleanup
|
test results on android:
|
- Permission request improvements
iaigner
left a comment
There was a problem hiding this comment.
LGTM - when you have fixed the problem with the id for the app usage, you are good to go
- Fixed an view foreground/background issue




Other changes: