Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Cron.cs - MinuteInterval & HourInterval #2141

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

staviloglu
Copy link

Updated MinuteInterval & HourInterval methods to make them available (not obselete) again.

Added defensive code against interval value to get rid of the problem described in #1041

@odinserj

Also related:
#1054
#1779

Updated MinuteInterval & HourInterval methods to make them available (not obselete) again.

Added defensive code against interval value to get rid of the problem described in HangfireIO#1041 

Also related:
HangfireIO#1054
HangfireIO#1779
@odinserj
Copy link
Member

Thanks @staviloglu, this is an interesting way of providing the feature while trying to avoid its potential problems. However I still believe that one day there's much better implementation of this with proper intervals based on duration and not on the Cron format, and I believe sometimes it's much better to leave pain points to have more motivation of fixing them in full.

@staviloglu
Copy link
Author

@odinserj I totally understand you but also know that many people still use these methods even if they are obselete and they may or may not be informed about the issue #1041.

This PR, if they check change-logs or update their version of Hangfire, would inform anyone who uses these methods.

I need to confess that I still want to use those methods with the proper values 😄

Anyway, thanks for your time and this awesome library, keep up the good work.

@Spiralis
Copy link

Spiralis commented Dec 3, 2024

Well, there's always this library: https://github.com/schyntax/cs-schyntax

Used this many years ago and loved the system. It is ported to various languages, all based on Schyntax: https://github.com/schyntax

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants