izivkov / CasioGShockSmartSync

Apache License 2.0
115 stars 12 forks source link

Clarify "Day Reminder" on README #46

Closed booherbg closed 1 year ago

booherbg commented 1 year ago

Is your feature request related to a problem? Please describe. On the README, it states:

The watch only supports all-day reminders. However, if the Google calendar event has a specific time, it will still be used as a day reminder on the watch.

I am not a current casio user so I don't know what a "day reminder" is :) Is this a daily alarm? Or is it a different kind of reminder that exists on the watch, such that there are two reminder types? Or is it something else?

Describe the solution you'd like I'd like this sentence to be either clarified (day reminder vs daily alarm) or a description of what this really means for the user.

The watch only supports all-day reminders. However, if the Google calendar event has a specific time, it will still be used as a day reminder on the watch. This means that the time information will not be transferred to the phone but will still create an all-day reminder on the watch.

(Is my update correct?)

Describe alternatives you've considered N/A

Additional context Thank you for all your hard work :)

booherbg commented 1 year ago

PS: If you just give me a thumbs up, I'll make a PR to update the docs according to my suggestion above.

booherbg commented 1 year ago

Also... for time-specific reminders, is it possible to append the time itself to the reminder?

"Dinner @ 5pm" for example, could show up as the name of a 5:00pm reminder that has the label of "Dinner"

izivkov commented 1 year ago

The idea of adding time to the reminder is good, and we can add it.

All your calendar events will show, but you can select up to 5, and these 5 will be sent to the watch.

For automatic time update, the calendar events will also be sent to the watch as reminders.

The watch can update automatically, as long as the app is running. We did not make it run forever in the foreground, because it is using too much battery, but you can use something like MacroDroid to keep it up.

On Wed., Aug. 2, 2023, 12:38 p.m. Blaine Booher, @.***> wrote:

Also... for time-specific reminders, is it possible to append the time itself to the reminder?

"Dinner @ 5pm" for example, could show up as the name of a 5:00pm reminder that has the label of "Dinner"

— Reply to this email directly, view it on GitHub https://github.com/izivkov/CasioGShockSmartSync/issues/46#issuecomment-1662563494, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7M37TAS4CYX2OCCDR55VLXTJ7B5ANCNFSM6AAAAAA3BRFARI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

booherbg commented 1 year ago

Thank you for the quick response.

In the case of using MacroDroid (good idea) will it automatically select the first 5? Like would there be any kind of manual intervention needed from my end, especially if there are less than 5 reminders for today?

izivkov commented 1 year ago

@booherbg : Yes, in automatic mode, the first 5 events will be sent to the phone. In interactive mode, the user can select which events to be sent to the watch, before sending. I thought of saving the selection to be used in auto-mode as well, but events can change, so simple selected index will not work. I don't want to overcomplicate things.

About the app staying up, and MacroDroid: At one time the app worked like the official Casio app, always keeping a notification on to stay running in the background. While that worked reasonably well, it was very annoying to see the app "always in my face". It also used more battery. So I moved away from this. In general, having user apps run constantly is not the Android way, and is discouraged by Google. Using MacroDroid works somewhat, but for perfect. I have a macro to start the app (If not running) 4 times a day, before the watch updates itself, and it worked, but not perfectly. So, your expectation of start-and-forget mode may not be fully achievable. BTW, I am considering re-introducing the always-run-in-the-background mode with notification for people who want to use it, but I'll make it optional. Most people use the app occasionally to set the time and timer, so they should not be "punished" with shorter battery life and annoying notifications.

Your idea of putting the time in the reminder is a good one, but my only concern is using up characters in an already very limited reminder message (I think it is 16 chars). Many of my Google events are already cut-off on the watch. Also, I think the time should be in front of the message, something like "@8:30am: Meeting with John". So, you see 9 characters are used up by the time display, and the message will be: "@8:30am: Meeting". So, I'll look into it a but further.

By all means, create a PR to update the docs. I see you are a developer, so if you have some ideas on adding features or fixing the app, drop me a line and contribute to the project.

izivkov commented 1 year ago

To be clear, the watch 5 has daily alarms, which can be set at specific time.

Also, it has 5 reminders, which can be set at a specific date, without specific time. These reminders are similar to Google's all-day events. They can be repeating on a particular weekday(s), month days, can have start and end date, The Casio reminders are not as advanced as the Google calendar's events. For example you cannot set a reminder on the second Tuesday of the month, but the app tries to implement as many features of the Google calendar as possible. For example, in Google calendar, you can set an event to repeat 4 times, staring on a particular date. Casio does not support that, but the app calculates the end date, and sets the reminder accordingly.

Hope this helps.

izivkov commented 1 year ago

Done

booherbg commented 1 year ago

Thank you for the great conversation. If I end up with one of these models I'll be sure to give good feedback.

izivkov commented 1 year ago

👍

On Tue., Aug. 8, 2023, 1:49 a.m. Blaine Booher, @.***> wrote:

Thank you for the great conversation. If I end up with one of these models I'll be sure to give good feedback.

— Reply to this email directly, view it on GitHub https://github.com/izivkov/CasioGShockSmartSync/issues/46#issuecomment-1668947642, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7M37QGRJFK6YBAJUTENFLXUHHPXANCNFSM6AAAAAA3BRFARI . You are receiving this because you modified the open/close state.Message ID: @.***>