Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show Battery Status on Battery Powered Devices #290

Closed
15 of 19 tasks
ghost opened this issue Mar 29, 2021 · 2 comments
Closed
15 of 19 tasks

Show Battery Status on Battery Powered Devices #290

ghost opened this issue Mar 29, 2021 · 2 comments

Comments

@ghost
Copy link

ghost commented Mar 29, 2021

Is your feature request related to a problem? Please describe.
It seems that serverinfo repo is not active so I submit it here.
I host my NC on a battery powered device. I have to mind the battery level for fear that it may go poweroff.
However, to make it more durable, it's headless and the X11 is not enabled by default, which means I need another solution to check the battery status.

Describe the solution you'd like

  • Display the battery capacity by reading /sys/class/power_supply/*/capacity.
    • Multi battery support? I have only mono battery devices so I cannot debug it.
  • Detect charging status by reading /sys/class/power_supply/*/status.
  • BSD support
    • Detect charging type. Yet to come up with a way to implement.
  • Monitor the battery usage by displaying a chart.
    • Prompt admins to enable Cron.
    • Cron background jobs.
    • Migration and creation of the new table oc_serverinfo_battery.
  • API support.
    • What sort of useful data should the API respond?
  • Warn admin when battery is low. Currently warns at the level of 50%.
    • How low? 25%? 50%?
    • Drop app.php and use new Application.php.
  • L10N support.
  • UI Improvements.
    • Make the chart pretty. Could use the help from an expert on UI design...
    • Currently using modified Gnome icons. Not sure if it's licensed.
    • Add a live battery icon that changes with the battery capacity.

Screenshot

0
1
2

@ghost ghost changed the title Show Battery Capatity on Mobile Instance Show Battery Capatity on Battery Powered Devices Mar 29, 2021
@ghost ghost changed the title Show Battery Capatity on Battery Powered Devices Show Battery Status on Battery Powered Devices Apr 8, 2021
@joshtrichards
Copy link
Member

While this response is going to a @ghost account, I'll comment for the record.

At first I thought this was solely a detailed request and a compelling visual mock-up, but it looks like you may have gotten pretty far along in your implementation. Unfortunately I don't see an associated PR. I hope you found what you needed on your journey!

I sympathize with your need, but I don't really see why this needs to be integrated into NC itself. I suspect there are some reasonable solutions that don't require being integrated with NC at all. Alternatively, you might be able to move your implemented functionality into a dedicated NC app. Lastly, I can't find any evidence that anyone else has ever requested a battery related feature for NC.

Tldr: IMO this falls outside the scope of NC/serverinfo. That said, I'm only one voice.

@kesselb kesselb closed this as not planned Won't fix, can't repro, duplicate, stale Jun 6, 2023
@kesselb
Copy link
Collaborator

kesselb commented Jul 15, 2023

This sounds like a great feature, but currently there are no plans to implement it. Therefore, I will close this ticket for now. This does not mean we don't want this feature, but it is not on our list of things to do soon. If someone wishes to implement this feature, we are happy to assist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants