Demizo / Daily_You

Every day is worth remembering...
GNU General Public License v3.0
124 stars 5 forks source link

New Feature: Random reminders #87

Open Ranger-NF opened 8 months ago

Ranger-NF commented 8 months ago

This PR includes the feature requested in #50. New button random reminders in settings menu.

How it works?

Every time the random reminders button is pressed, app calculates completely random time and sets an alarms using existing system.

Limitations:

Once set, the time will be same everyday (Although it can be changed by pressing the above mentioned button again).

I wasn't able to come up with a good solution for this without making it complicated.

Ranger-NF commented 8 months ago

Forgot to pull latest changes. Let me fix it

Demizo commented 8 months ago

Awesome work! It may be a bit before I merge this in as I want to give some thought to how to do the following:

Ranger-NF commented 8 months ago

@Demizo Im having troubles when building the latest code

Its showing errors related to shared-storage...

Logs:


../../../.pub-cache/git/shared-storage-789927b69de53bb4dbe5b4b803da0e5944f9bf40/lib/src/common/functional_extender.dart:28:36: Error: A value of type 'Object?' can't be returned from a function with return type 'T?'.
 - 'Object' is from 'dart:core'.
    return self != null && f(self) ? self : null;
                                   ^
Target kernel_snapshot failed: Exception

Do you know any solutions?

Ranger-NF commented 8 months ago

It may be a bit before I merge this in as I want to give some thought to how to do the following

Oh okay, thats fine by me. Do you want me to close this PR?

Demizo commented 8 months ago

@Ranger-NF Odd, that package is in the process of being overhauled by the main developer so it may have just fallen out of date. Can you try lowering your flutter SDK version to 3.16.0 again? It looks like this branch is on 3.18.0.

Demizo commented 8 months ago

@Ranger-NF No feel free to leave it open! If I come up with those changes I will use this PR as a base so you get credit for the merge 😊

Ranger-NF commented 8 months ago

Okay, it should be working properly now