-
-
Notifications
You must be signed in to change notification settings - Fork 52
feat: Handle paused state + workflow performance improvements #2610
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
Conversation
afcf67f
to
8ed6c60
Compare
8ed6c60
to
7487e61
Compare
971531e
to
d14945f
Compare
7487e61
to
52ccd61
Compare
52ccd61
to
44dccad
Compare
417ac1d
to
b5891b3
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like the updated execution time formatter will render <0 minutes
when the duration is 0. I believe detailsRelevant
needs to be taken into account when considering when to include the "<"
in compactMinutes
.
Chatted on Discord, gonna just swap out the three dots I think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
b7239d1
to
757ea12
Compare
Resolves #2569
Changes
paused
workflow state.Task
to improve poll performance.Manual testing
Screenshots