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.

Sign up for GitHub

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

Jump to bottom

fix(cron): Ignore time sensitivity when a class was explicitely scheduled #50942

Merged
merged 1 commit into from
Feb 21, 2025
Merged

fix(cron): Ignore time sensitivity when a class was explicitely scheduled #50942

merged 1 commit into from
Feb 21, 2025

Conversation

nickvergessen
Copy link
Member

Checklist

@nickvergessen
fix(cron): Ignore time sensitivity when a class was explicitely sched…
72e3e13 
…uled

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added bug 3. to review Waiting for reviews labels Feb 21, 2025
@nickvergessen nickvergessen added this to the Nextcloud 32 milestone Feb 21, 2025
@nickvergessen nickvergessen self-assigned this Feb 21, 2025
@kyteinsky
Copy link
Contributor

can we add an explicit arg like -f to ignore the time sensitivity?
This will also keep the old behaviour in case someone/thing uses that.

@nickvergessen nickvergessen mentioned this pull request Feb 21, 2025
Merged
4 tasks
@kyteinsky
Copy link
Contributor

there might not be so many people relying on the niche behaviour

@nickvergessen nickvergessen merged commit 6cd3046 into master Feb 21, 2025
187 of 189 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/ignore-sensitivity-when-explicitly-scheduled branch February 21, 2025 13:40
@nickvergessen
Copy link
Member Author

/backport to stable31

@backportbot backportbot bot mentioned this pull request Feb 21, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcelklehr marcelklehr marcelklehr approved these changes

@kyteinsky kyteinsky kyteinsky approved these changes

Assignees

@nickvergessen nickvergessen

Labels
3. to review Waiting for reviews bug
Projects
None yet
Milestone
Nextcloud 32
Development

Successfully merging this pull request may close these issues.
3 participants
@nickvergessen @kyteinsky @marcelklehr