Skip to content

Conversation

@githtz
Copy link

@githtz githtz commented Feb 2, 2018

What does this do
Add option to hold the current battery level when the device gets plugged in and battery level is above the lower limit and below the upper limit.
If the option is disabled the current behaviour of the app should be just like now.
If the option is enabled the device will start recharging only if the battery level is lower then the lower limit specified in the application settings.

Why could this be useful
This might be useful when the device gets plugged in and out multiple times a day, for example when the device is used with external display and input devices. Prevents the battery from being charged over and over again.

Further suggestions are very welcome

@sriharshaarangi
Copy link
Owner

@githtz Sorry for the late response.
Actually, I want to make that as default behaviour. But before that, I want to restrict the lower threshold, as some people are keeping lower threshold very low(below 30) as they don't seem to understand lower threshold is needed to maintain charge in that range when plugged in.

@githtz
Copy link
Author

githtz commented Sep 4, 2018

No problem at all, I'm quite happy to hear this will become a part of your app in future.

…yLevel

# Conflicts:
#	app/src/main/res/values-de/strings.xml
@sriharshaarangi sriharshaarangi force-pushed the master branch 2 times, most recently from 51c5b58 to 997350f Compare February 9, 2019 18:18
…yLevel

# Conflicts:
#	app/src/main/java/com/slash/batterychargelimit/activities/MainActivity.kt
#	app/src/main/java/com/slash/batterychargelimit/receivers/BatteryReceiver.kt
#	app/src/main/res/layout/content_main.xml
#	app/src/main/res/values/strings.xml
@githtz
Copy link
Author

githtz commented Feb 19, 2020

@sriharshaarangi @milux @xdnax Maybe this could be merged some time soon? I've once more fixed all merge conflicts 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants