Fix "Rings On" feature - correct time display for future date alarms Implement Mutually Exclusive Scheduling Options for Alarms #830
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes an issue with the "Rings On" feature where the time until the alarm was not correctly displayed for alarms set to future dates. The home screen and alarm settings page now correctly show the proper duration (e.g., "2 days" for an alarm set 2 days in the future).
This PR implements a new scheduling interface that makes alarm scheduling more intuitive by ensuring that "Ring On" (one-time) and "Repeat" (recurring) options are mutually exclusive.
Proposed Changes
-Enhanced timeUntilAlarm() method in Utils class to handle future date alarms with ringOn flag
-Added formatted duration display for multi-day alarms
-Updated all relevant controllers to pass the ringOn and alarmDate parameters
-Added proper time updating when toggling the "Ring On" feature
SchedulingOptionsTile
with a tabbed interface for selecting between one-time and recurring alarmsFixes #827
Screenshots
WhatsApp.Video.2025-04-25.at.02.49.58.mp4
Checklist