Skip to content

Update Cron.cs - MinuteInterval & HourInterval#2141

Open
staviloglu wants to merge 2 commits intoHangfireIO:devfrom
staviloglu:master
Open

Update Cron.cs - MinuteInterval & HourInterval#2141
staviloglu wants to merge 2 commits intoHangfireIO:devfrom
staviloglu:master

Conversation

@staviloglu
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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