-
Notifications
You must be signed in to change notification settings - Fork 780
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
Add inbound parse webhook example #792
Conversation
17885b1
to
0cb87a6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great! I may have missed it, but left a comment in regards to another section in the AC for the issue covered here.
@@ -0,0 +1,21 @@ | |||
# Introduction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In reference to the AC, we'll want some documentation around deployment to a cloud service and how to start processing events.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, let me see what I can do there. I was just using ngrok to test.
@reedsa I don't have a cloud environment at the moment to document everything in a cloud deployment. I added Kubernetes manifests and documented how to deploy using Kubernetes running locally. |
Hello @aroach, |
@aroach Awesome work on the docs! Thanks for the extra effort on this one, I know there was a lot to take care of. 💯 👍 Not certain if another issue should be created to cover the deployment specific stuff at a later time? |
@reedsa let me see if I can get setup for something to document it
tomorrow.
…On Tue, Oct 9, 2018 at 8:53 PM Elmer Thomas ***@***.***> wrote:
Merged #792 <#792> into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#792 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACL-snWtWVYFovuDC_VB4E4WLqgImqQtks5ujWFAgaJpZM4XNytF>
.
|
Fixes
#787
Checklist
Short description of what this PR does:
If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.