Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Power type reported as "Wired" while on battery #833

Closed
3 of 10 tasks
garlic-os opened this issue Nov 4, 2020 · 5 comments · Fixed by #910
Closed
3 of 10 tasks

Power type reported as "Wired" while on battery #833

garlic-os opened this issue Nov 4, 2020 · 5 comments · Fixed by #910

Comments

@garlic-os
Copy link

Technical information

Using version:

  • master (running from GitHub-published source code, currently v3.0.0-pre)
  • latest (latest release, currently v2.2.2)
  • vX.X.X (specify other version)

Running on:

  • Linux
  • Windows
  • macOS

How comfortable you are with your system and/or IT in general:

  • I'm kind of lost, honestly
  • I know what's up, I could help you run some commands or checks
  • My machine is fully under my control, tell me what you need
  • I attended Defcon last year

Problem

I'm using a Lenovo laptop. While it's on battery, if I start up Edex, it will still say that my power status is "Wired", when I would expect it would say something like "Battery" instead.

My laptop is a Lenovo L380 Yoga TP00019A.

@parkerlreed
Copy link

Please test against master

#769

@wrac4242
Copy link
Contributor

I have this issue on master branch, running on a windows laptop

@GitSquared
Copy link
Owner

Sounds a lot like sebhildebrandt/systeminformation#145, but this was fixed years ago... Will check this out.

@wrac4242
Copy link
Contributor

done some testing, the main code to check this is here, in line 125,
} else if (bat.acconnected || bat.timeremaining === -1) {

the issue is caused by bat.timeremaining I belive
due to it being the only thing set, and when removed correctly displays battery percentage

@GitSquared
Copy link
Owner

mind submitting a PR?

GitSquared pushed a commit that referenced this issue Nov 23, 2020
@lebarde lebarde mentioned this issue Nov 25, 2020
18 tasks
eugene2candy pushed a commit to eugene2candy/edex-ui that referenced this issue Apr 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants