Skip to content

Feat/realtime webhook validation#704

Open
michaelvic123 wants to merge 3 commits into
SoroScan:mainfrom
michaelvic123:feat/realtime-webhook-validation
Open

Feat/realtime webhook validation#704
michaelvic123 wants to merge 3 commits into
SoroScan:mainfrom
michaelvic123:feat/realtime-webhook-validation

Conversation

@michaelvic123

Copy link
Copy Markdown

This PR adds:

  • Real-time validation : Checks URL format and HTTPS requirement as user types, not just on blur
  • HTTPS enforcement : Disallows HTTP URLs entirely
  • Test Connection button : Lets users verify webhook URLs work before saving (requires backend mutation testWebhookUrl )
  • Better error messages : Clear error messages for invalid/non-HTTPS URLs
  • Updates timeout validation : Timeout field also shows errors in real-time
    Changes made:
  1. CreateWebhookModal.tsx : Add test connection button, real-time validation
  2. WebhookManager.tsx : Add real-time validation, test connection button
  3. webhooks/[id]/page.tsx : Update timeout field to validate in real-time
  4. django-backend/schema.py : Add test_webhook_url GraphQL mutation
  5. webhook-components.test.tsx : Update test to match new error message
  6. Fix linter warnings for unused variables

closes #573

- Real-time URL format validation as user types
- HTTPS enforcement
- Add test connection button
- Add error messages for invalid URLs
- Update timeout validation to also be real-time
@drips-wave

drips-wave Bot commented Jun 1, 2026

Copy link
Copy Markdown

@michaelvic123 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@DokaIzk

DokaIzk commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

@michaelvic123 please resolve conflicts and ensure checks pass

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.

Webhook URL Validation UI

2 participants