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

Feature/webhooks #38

Merged
merged 4 commits into from
May 13, 2019
Merged

Feature/webhooks #38

merged 4 commits into from
May 13, 2019

Conversation

mihok
Copy link
Member

@mihok mihok commented May 13, 2019

Description

We're adding the concept of webhooks into the mix, now if you want to be notified of a new client or new operator, or new client message you can register a webhook and the daemon will attempt to hit the endpoint similar to how Stripe's webhook functionality works.

Motivation

There currently is no notification system that allows for someone to be notified when the operator app is not open of incoming messages, webhooks solves this without having to build a direct notification system by integrating into whatever folks already use for mail.

Changes

  • New Webhook and Event protocol buffer objects defined
  • Moved files/folders to update with how Golang typically has them
  • Fixed the linting make commant, added comments and fixed a great many linting errors in the process

@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling f59aba6 on feature/webhooks into b2db56c on master.

@mihok mihok merged commit 582a8aa into master May 13, 2019
@mihok mihok deleted the feature/webhooks branch May 13, 2019 21:56
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.

None yet

2 participants