Meta-issue: Handle disabled sensors in a generic way
aaaaalbert opened this issue · 3 comments
@ReyhanJB reports in issues #17, #18, #19 that Sensorium keeps trying to use sensors even if the user disabled them through Android's (but not the app's) configuration screens. Sensorium's AbstractSensor class deals with properties, data, and methods that are generic across different sensor types.
It may make sense to add a generic way to auto-disable such turned-off sensors to AbstractSensor (and also re-enable them as fit) --- perhaps a configurable BroadcastReceiver that you configure for a specific intent relates to turning a sensor on or off, and an additional latch that enables/disables Sensorium's other actions for this sensor.
@aaaaalbert @fmetzger I made some changes to the source code to address the issues #17, #18, and #19. How can I push the changes? It requires authentication.
That's great to hear!
However, I'd ask you to create and submit individual pull requests for each of the issues so that we can review and merge them.
Done.