Skip to content

Conversation

@timliang4
Copy link
Contributor

@timliang4 timliang4 commented Nov 16, 2025

Summary

Adds integration with Windmill's webhook service to enable notification delivery (SMS, etc.) from the backend.

Changes

  • Created Notification package with WindmillNotificationClient
  • Client uses OkHttpClient to send webhook requests to Windmill
  • Reads Windmill and Twilio credentials from environment variables

Usage

  • Should be injected as singleton through AppConfig for efficiency (HTTP client reuse)
  • Once injected into a service, it can call client.sendSms(phoneNumber, message) to trigger notifications via Windmill webhook.

Testing

  • Unit tests
  • Manual test for sending actual webhooks

TODO

  • Make notification flow available organization-wide on Windmill
  • Set up webhook token for backend (currently using a token associated with my Windmill account)

@timliang4 timliang4 requested a review from crchong1 November 16, 2025 22:25
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