Skip to content

fix: startedDate and finishedDate should be required#7553

Merged
Startrekzky merged 1 commit into
mainfrom
kw-webhook-deployment-required
May 31, 2024
Merged

fix: startedDate and finishedDate should be required#7553
Startrekzky merged 1 commit into
mainfrom
kw-webhook-deployment-required

Conversation

@klesh
Copy link
Copy Markdown
Contributor

@klesh klesh commented May 31, 2024

Summary

fix: startedDate and finishedDate should be required according to @louis

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. component/plugins This issue or PR relates to plugins severity/p1 This bug affects functionality or significantly affect ux type/bug This issue is a bug labels May 31, 2024
Copy link
Copy Markdown
Contributor

@Startrekzky Startrekzky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 31, 2024
@Startrekzky Startrekzky merged commit afd1763 into main May 31, 2024
@Startrekzky Startrekzky deleted the kw-webhook-deployment-required branch May 31, 2024 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins lgtm This PR has been approved by a maintainer severity/p1 This bug affects functionality or significantly affect ux size:XS This PR changes 0-9 lines, ignoring generated files. type/bug This issue is a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants