search

itachi1706 / GI-Weekly-Material-Tracker

A flutter based application that works with Firebase to provide material tracking for Genshin Impact
2 stars 1 forks source link

Daily Reminder to do forum daily login event #287

Closed itachi1706 closed 2 years ago

itachi1706 commented 3 years ago

An option in the settings to toggle that when toggled, will inform you to do daily forum logins every day

This will be easy to calculate as it is all GMT+8, so we just have to set the reminder at 12 midnight GMT+8.

This reminder is in the form of notifications, where each notification will provide a button to launch the browser straight into the website to claim directly

Event URL: https://webstatic-sea.mihoyo.com/ys/event/signin-sea/index.html?act_id=e202102251931481&lang=en-us

itachi1706 commented 3 years ago

Use this for scheduling notifications https://pub.dev/packages/flutter_local_notifications#scheduling-a-notification

itachi1706 commented 3 years ago

changed time estimate to 4h

itachi1706 commented 3 years ago

We should also add a notification debug menu that's only visible during debug mode which will allow us to send the notifications that we will be scheduling directly. This allows us to do testing for the notification

This menu will be shown in the settings page

itachi1706 commented 3 years ago

https://medium.com/@fuzzymemory/adding-scheduled-notifications-in-your-flutter-application-19be1f82ade8

https://pub.dev/packages/flutter_local_notifications

itachi1706 commented 3 years ago

Might look at using https://pub.dev/packages/awesome_notifications?w=1 over flutter_local_notifications as well as the former does not seem to have action buttons support

itachi1706 commented 3 years ago

After some analysis, we might just go ahead and try out with awesome_notifications, however we will need to properly test and make sure we are able to do so. If it is not stable, we will fallback to flutter_local_notifications and make use of users tapping on the notification as the action instead of through buttons with the following link

https://pub.dev/packages/flutter_local_notifications#initialisation

The only real reason is that we do not exactly need the notifications for web, and we are not compiling for iOS nor macOS yet. Hence as long as Android works it would be perfectly fine already (web working is a benefit though). However it needs to fully support all we need to do (schedule when enabled, cancel when disabled and actually displaying the thing)

itachi1706 commented 3 years ago

created merge request !107 to address this issue

itachi1706 commented 3 years ago

mentioned in commit 1bf7621815b76b9f650651ae22417019ad97e1d3

itachi1706 commented 3 years ago

mentioned in commit 699283dd7cec84ea7c16a1074874a1173d4d368b

itachi1706 commented 3 years ago

mentioned in commit 5fe2f6de3183c23fc53dc7a8426a0d6693108c80

itachi1706 commented 3 years ago

mentioned in commit 1b54434eb4d5c62cfd1f10d161f2926866d5b564

itachi1706 commented 3 years ago

mentioned in commit fd1f408188eaf002ad35c3e5ee02a66f28a3d56e

itachi1706 commented 2 years ago

mentioned in commit 6de15b0fcf37ad91f8f357af825548ea993d9db6