Feat/reminder interval

[bugrakocabay]

Hi @TwiN

Summary

Regarding this issue #599.

Checklist

• Tested and/or added tests to validate that the changes work as intended, if applicable.
• Updated documentation in README.md, if applicable.

[TwiN]

Few comments, but looks good overall.

[illia-weisser]

I want this!

[TwiN]

Wow I'm really sorry for forgetting about this for so long!

@bugrakocabay any chance you can resolve the conflict and ping me when it's done? I'll get this merged and released ASAP

[TwiN]

Can this be renamed to MinimumRepeatInterval?

Because it doesn't necessarily happen every repeat interval given that the evaluation is driven by the normal interval. For instance, if the interval is set to 5m and the repeat interval is set to 7m, the alert would only be sent on the 2nd evaluation (so after 10m), not after the repeat interval of 7m

[sachasmart]

This would be great!

[nosovk]

@bugrakocabay could you please merge bugrakocabay#1, I hope its only a one obstacle to merge this.

[saul-salazar-dotcom]

thanks for this feature! hope it gets merged soon 🥇

[ViktorZieg]

Any chance this will be merged anytime soon?

[onedr0p]

@bugrakocabay could you please merge bugrakocabay#1, I hope its only a one obstacle to merge this.

@nosovk This was merged into the wrong branch, so it's not reflected here. @bugrakocabay this needs that to be addressed and also rebased. I'll wait a little while and if not done will fork your fork, make the required changes and open a new PR so we can try to get this moving forward.

[onedr0p]

Actually looks like we can close this in favor of #1138, all authors have been carried over.

@TwiN what do you think?

[TwiN]

Good point @onedr0p

Closing in favor of #1138

[TwiN]

#1138 has been implemented.

If some of you are still interested in the feature, I'd love a confirmation that it works as intended on your end.

Should only be available under the latest tag for now.