direc85 / harbour-batterybuddy

Battery level alert for Sailfish OS
GNU General Public License v3.0
7 stars 8 forks source link

PoC/RfC: read and display battery health and temperature #40

Closed nephros closed 3 years ago

nephros commented 3 years ago

Following up on my wish from https://openrepos.net/comment/37359#comment-37359 this is a first step to handling battery temperature and health status.

Shows temp and health on the main page.

direc85 commented 3 years ago

Wow, that's great! I was going to make a new issue for this, but I ended up finding a PR instead!

Looks good indeed, I'll have to give this one a proper spin on all my devices first :) I would like to see this in daemon side with a fresh set of notifications, too, but first things first.

I'll try to create a branch for this and merge it manually there, so this can be developed separately. Let's see how that goes...

nephros commented 3 years ago

Wow, that's great! I was going to make a new issue for this, but I ended up finding a PR instead!

Looks good indeed, I'll have to give this one a proper spin on all my devices first :) I would like to see this in daemon side with a fresh set of notifications, too, but first things first.

I'll try to create a branch for this and merge it manually there, so this can be developed separately. Let's see how that goes...

:)

I've started working on the rest (settings, service/daemon) over here: https://github.com/nephros/harbour-batterybuddy/tree/health-monitor. If you prefer I can add you as dev there so you don't need to branch here, whichever is better.

direc85 commented 3 years ago

I threw myself in the loop for a while, because I had merged the copypaste fix first, but there's now a health branch, so please push any future work into that branch. Thanks :)