Skip to content

[WIP] Remove Webhook#22617

Draft
RensR wants to merge 6 commits into
developfrom
rm-webhook
Draft

[WIP] Remove Webhook#22617
RensR wants to merge 6 commits into
developfrom
rm-webhook

Conversation

@RensR
Copy link
Copy Markdown
Contributor

@RensR RensR commented May 22, 2026

This PR removes the Webhook job, but keeps the db tables and the UI elements, but removes all webhook job logic.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

CORA - Pending Reviewers

Codeowners Entry Overall Num Files Owners
* 13 @smartcontractkit/foundations, @smartcontractkit/core
/.changeset/ 1 @smartcontractkit/foundations, @smartcontractkit/core
/core/services/job/ 3 @smartcontractkit/foundations, @smartcontractkit/core
/core/services/webhook/ 10 @smartcontractkit/foundations, @smartcontractkit/bix-framework, @smartcontractkit/core
/core/web/resolver/ 4 @smartcontractkit/foundations, @smartcontractkit/core
/deployment/ 1 @smartcontractkit/ccip-tooling, @smartcontractkit/ccip-offchain, @smartcontractkit/keystone, @smartcontractkit/operations-platform, @smartcontractkit/core
/docs/CONFIG.md 1 @smartcontractkit/foundations, @smartcontractkit/core, @smartcontractkit/devrel

Legend: ✅ Approved | ❌ Changes Requested | 💬 Commented | 🚫 Dismissed | ⏳ Pending | ❓ Unknown

For more details, see the full review summary.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

✅ No conflicts with other open PRs targeting develop

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented May 22, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
TestIntegration_AsyncEthTx/with_FailOnRevert_disabled,_run_succeeds_with_output_being_reverted_receipt Logs ↗︎
TestIntegration_AsyncEthTx/with_FailOnRevert_enabled,_run_fails_with_transaction_reverted_error Logs ↗︎
TestIntegration_AsyncEthTx Logs ↗︎
TestIntegration_AsyncEthTx/with_FailOnRevert_disabled,_run_succeeds_with_output_being_reverted_receipt The test failed without a specific error message, but the logs indicate a potential issue with the asynchronous Ethereum transaction or its revert... Logs ↗︎

... and 12 more

View Full Report ↗︎Docs

@cl-sonarqube-production
Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required ≥ A)
19.38% Technical Debt Ratio on New Code (required ≤ 4%)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube IDE SonarQube IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant