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

Change: Disable start icon for tasks with duration limit #3918

Merged
merged 3 commits into from
Nov 23, 2023

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Nov 22, 2023

What

If a task has a schedule with a duration limit assigned, the start icon will be disabled.

Why

This is done because there is currently no distinction between a scheduled start and a manual one, so the schedule would often stop the task immediately.

References

GEA-379

Checklist

  • Tests

If a task has a schedule with a duration limit assigned, the start icon
will be disabled.

This is done because there is currently no distinction between a
scheduled start and a manual one, so the schedule would often stop the
task immediately.
Copy link

Conventional Commits Report

Type Number
Changed 1

🚀 Conventional commits found.

Copy link

codecov bot commented Nov 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (248936e) 59.98% compared to head (c8bb273) 59.99%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3918   +/-   ##
=======================================
  Coverage   59.98%   59.99%           
=======================================
  Files        1023     1023           
  Lines       25173    25177    +4     
  Branches     7470     7473    +3     
=======================================
+ Hits        15100    15104    +4     
  Misses       9125     9125           
  Partials      948      948           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@timopollmeier timopollmeier marked this pull request as ready for review November 22, 2023 13:35
@timopollmeier timopollmeier requested a review from a team as a code owner November 22, 2023 13:35
@bjoernricks bjoernricks merged commit 9cf69e1 into main Nov 23, 2023
13 checks passed
@bjoernricks bjoernricks deleted the disable-limited-duration-task-start branch November 23, 2023 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants