J. Dekker
J. Dekker
@anthraxx what did you switch to?
I'll give implementing it as a separate face a go, but I'm concerned about divergence with the simple_clock_face since it should have feature parity apart from the extra audible signals....
@joeycastillo what do you think of this? Essentially just simple_clock with the variable beep feature inside the face itself.
Hi, I'm looking at this again. @WesleyAC, if the face is only a settings face managing the granular beep, wouldn't this conflict with simple_clock_face's hourly signal? I'm not understanding how...
@matheusmoreira any thoughts on this? could it be integrated into the new clock_face.c or where should we go from here
@matheusmoreira sure, for the simple case, but I'd really like this to be configurable at runtime (the entire functionality can be build time `#ifdef` I don't mind with that though)....
Honestly, I'm fine for it to go in as unconfigurable until you rework the preferences (I have two sensor watches, so I'll just carry one which each interval I use)....
@matheusmoreira apologies for my delay on this. Updated and force pushed to the PR as https://github.com/joeycastillo/Sensor-Watch/pull/279/commits/0336a0a74d5fb0269099f6ef46406527d1affb48.
@drdv can you squash this into one commit? cc: @bcc32
@league is this still required?