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

fix: [TKC-2209] add webhooks events #5666

Merged
merged 18 commits into from
Jul 19, 2024
Merged

Conversation

vsukhin
Copy link
Collaborator

@vsukhin vsukhin commented Jul 16, 2024

Pull request description

  • deprecate onStateChanges field for webhook
  • add new event types for state machine changes for Tests, Suites, Workflows
  • support aggregation event types

Checklist (choose whats happened)

  • breaking change! (describe)
  • tested locally
  • tested on cluster
  • added new dependencies
  • updated the docs
  • added a test

Breaking changes

Changes

Fixes

Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Copy link

netlify bot commented Jul 16, 2024

Deploy Preview for testkube-docs-preview ready!

Name Link
🔨 Latest commit ba12beb
🔍 Latest deploy log https://app.netlify.com/sites/testkube-docs-preview/deploys/669a595cb5983600082ef18a
😎 Deploy Preview https://deploy-preview-5666--testkube-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin vsukhin changed the title fix: [TC-2209] add webhooks events fix: [TKC-2209] add webhooks events Jul 16, 2024
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin vsukhin marked this pull request as ready for review July 18, 2024 15:22
@vsukhin vsukhin requested review from a team as code owners July 18, 2024 15:22
Signed-off-by: Vladislav Sukhin <[email protected]>
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin vsukhin merged commit af6ac02 into develop Jul 19, 2024
11 checks passed
@vsukhin vsukhin deleted the vsukhin/fix/add-webhooks-events branch July 19, 2024 12:36
vsukhin added a commit that referenced this pull request Jul 23, 2024
* fix: sync docs

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: remove on state changed

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: unit tests

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: add new webhook events

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: remove link

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: remove slack events

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: support new evenys in webhooks

Signed-off-by: Vladislav Sukhin <[email protected]>

* docs: new event types

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: unit test for event types

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: unit tests for events

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: unit tests for exetended event type

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: add parallel unit tests

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: unit tests for emitter

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: add break

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: add nil check

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: test suite event end time

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: dep update

Signed-off-by: Vladislav Sukhin <[email protected]>

* fix: doc typo

Signed-off-by: Vladislav Sukhin <[email protected]>

---------

Signed-off-by: Vladislav Sukhin <[email protected]>
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