Comments (22)
OK, thanks. Found it and deactivated it with succes.
Not the best option/solution for an "average" user though.
Robby
from androidaps.
the reason why I added notification channel is that in logs were a message that communication service was killed because it didn't call startforeground. That disappeared after mentioned change. My thoughts is that complications cannot work properly without this service.
The change what started this is targeting API 29 on watch in last master version
from androidaps.
Confirming:
See also #3137 (AAPS Wear: Ongoing bolus notification that can not be cleared)
This commit may be fixing a problem for a minority of users not running a 3th party watch face, but is apparently causing problems for most users running AAPS-wear the way it was designed for?
Proposed solution:
Rollback changes and review for better solution,
or alternatively: enable "Wear: Foreground Service" only as a user option through watch face settings (default = OFF)
from androidaps.
Till now, the notifications were working great . Now we're stuck with a notification that we can't erase.
The fix made things worst (at least for some (or many ?) users).
Used on Android 13 and 14.
from androidaps.
totally against more options :)
from androidaps.
You're the boss and I totally agree on there are too many options syndrome (ie xDrip) ;)
How many people use the AAPS watchface and how many use others ?
I still believe that more will be bothered with the change than happy to gave it.
Of course, you can ignore the stuck notification on the watch but I'll be the one to tell them to ignore it when they'll complain about it on Discord... :(
from androidaps.
totally against more options :)
Agree. 'need optional option to make options optional :-)
from androidaps.
Turning OFF the notification turned OFF ALL notifications on watch from AAPS.
d75c465 Is not working as it should.
from androidaps.
Updated to the latest Dev on phone and watch tonight and the problem is still there.
from androidaps.
on android 10+ ongoing notification must be in place if you want to have service running on background
from androidaps.
in 3.2.0.3 service was suspended without active watchface (on background) and complications were not working. if you add notification android allows service running on background. the price for it is permanent notification
from androidaps.
in 3.2.0.3 service was suspended without active watchface (on background) and complications were not working. if you add notification android allows service running on background. the price for it is permanent notification
@MilosKozak @robertrub How about making this notification feature optional (read: configurable on/off)?
I could do a PR for this...
from androidaps.
Totally vote for it being an option.
from androidaps.
@MilosKozak As far as I know, using an ongoing notification is not required, although the system might enforce that depending on the Android version. It might also be worth an attempt to use PRIORITY_MIN. These notifications did not appear on my watch at all and I got a notification by the system instead that AndroidAPS was running in the background (which can be dismissed).
Anyways, on my Pixel Watch 2 I'm able to disable the notification channel in the system settings. Perhaps that setting is hidden on watches from other manufacturers. Manually launching the intent might work (see https://stackoverflow.com/questions/32366649/any-way-to-link-to-the-android-notification-settings-for-my-app). There's probably also a way to manage notifications via ADB.
from androidaps.
...] Perhaps that setting is hidden on watches from other manufacturers [...
With current dev-builds, on the Samsung Galaxy Watch 4, I can hide the ongoing notification through developer menu settings on Watch:
App Notifications -> All watch apps -> AAPS -> Data layer Foreground Service Channel: OFF
from androidaps.
No, I don't have that option. Samsung Android 14. Both phone and watch are set to developer mode.
Am I searching in the wrong place ?
from androidaps.
No, I don't have that option...
Wrong place. Check developer options menu on Watch:
- Scroll all to way down to "App Notifications"
- Select: All watch apps -> AAPS -> Data layer Foreground Service Channel
- Switch it to OFF
(btw: No need to set the Phone to developer mode)
from androidaps.
@MilosKozak Here are my thoughts on the matter. I don't think we want to solve a problem for a few users, and create a new problem that will concern all. As it stands now the data layer foreground notification is an annoyance that most users will have issues with. It vibrates often without any reason, and if you enact anything from the tiles it also comes up first, before what you usually see. This is tested with a Samsung Galaxy Watch 4 with latest dev and updates from google on watch.
I've been trying to alter the code to make it less annoying, it seems that the following helps, keeping it from blocking wanted notifications when enacting from tiles and vibrating alot less. It still isn't possible to remove tho, and I find that troubling, as it very often adds a dot to the watch screen making it look like you have a missed notification. Which is usually the datalayer foreground notification :/
I have tested a few days without commit 041d0f0 to pursue @TebbeUbben claim that a notification channel is not necessary, and so far, using stock samsung galaxy watch watchface I'm still able to enact from all commands from every tile. So maybe a persistent channel isn't necessary? Would be nice if more could test this, so we don't add stuff that isn't really necessary and just is in the way of notifications you really want to see.
Edit: Having issues when using AAPS watchface. So this is probably not a solution to directly revert that commit.
from androidaps.
I have tested a few days without commit 041d0f0
Originally had the idea to disable the changes based on a watch config option setting.
However, when I disable this commit in current dev by commenting out the changes (see code from the first screenshot) somehow this leaves the wear app unstable in that what looks like the watchface restarting about every minute or so?
Did not get to load this into the debugger (need to figure out how to do that ... ) to see what it is doing...
from androidaps.
I have tested a few days without commit 041d0f0
Originally had the idea disable the change based on a watch config option setting.
However, when I disable this commit in current dev by commenting out the changes (see code from the first screenshot) somehow this leaves the wear app unstable in that what looks like the watchface restarting about every minute or so? Did not get to load this into the debugger (need to figure out how to do that ... ) to see what it is doing...
Edit: Experiencing same problem as @vanelsberg using AAPS watchface when reverting commit 041d0f0
I had tested without AAPS watchface, and not enough with AAPS watchface.. :)
from androidaps.
the reason why I added notification channel ...
Thanks for clarifying, which would explain.
Seems solving this is not as straightforward as I anticipated.
from androidaps.
Please test latest dev 8cfa312 and see if it's better now. At least for me it works great. (Samsung Galaxy watch 4)
from androidaps.
Related Issues (20)
- AAPS logfiles no longer available on user phone storage (e.g: /AAPS/logs) HOT 4
- AAPS 3.3.0-dev-b crashes when trying to use quick wizzard HOT 7
- 3.3 dev-b User entries duplicated for Carbs HOT 3
- java.lang.NumberFormatException: empty String in Full Synchronization of Treatments HOT 2
- Onmipod Dash Low Reservoir alert silence on bolus ?
- MEDTRUM Nano disconnecting HOT 15
- Use local http webserver to receive data HOT 2
- Feature request: Move automation user button next to treatments HOT 5
- Minimum insulin action time can't be set less then 5.0 HOT 2
- my app not found closed loop HOT 1
- Autosens value on AAPS Client never goes above 120% with 1.5 upper sensitivity ratio on AAPS Master HOT 1
- [Suggestion] export settings from maintenance go to dowloads folder or straight to google drive HOT 3
- AAPS turns off Bluetooth HOT 6
- Data does not upload to Nightscout after switch the Insulin type to Lyumjev HOT 4
- NSv3 client connection - WiFi SSID can only be defined or changed, not cleared. HOT 1
- 3.3.0-dev Build failure: items queued for execution but none of them can be started HOT 6
- SMB Always and after carbs disabled HOT 2
- Wear Sleep Mode is Black on Native Watch Face HOT 2
- ComboV2 TBR duration 24 hours is not recognized HOT 1
- Feature / Medtrum Nano : add history HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from androidaps.