remove Event Delivery references #5049
Conversation
spencerattick
requested review from
forstisabella
and removed request for
a team
|
Hi @spencerattick - looks like in the thread this is currently unsupported but will end up being supported in GA. Could we keep it in the docs as a comment, and then surface this info again after Insert Functions goes GA? |
|
Hey @forstisabella! Would keeping it in a comment keep it out of the public docs for now? If so then that sounds great! :) I'm not sure how to do that though. 😅 |
|
@spencerattick yep - it remains in the file but doesn't render for an end user to see! The site honors HTML comment formatting, so a comment could look like this: <!--- I am a comment! --->Or you can use the Liquid comment format: {% comment %}
I am a comment!
{% endcomment %}You can wrap a both kinds of comment formats around multiple lines of code. |
| @@ -237,12 +237,6 @@ Note the following limitations for batching with insert functions: | |||
|
|
|||
| {% endcomment %} | |||
|
|
|||
There was a problem hiding this comment.
| {% comment %} |
| A function can throw errors, or Segment might encounter errors while invoking your function. You can view these errors in the [Event Delivery](/docs/connections/event-delivery/) tab for your Destination as in the example below. | ||
|
|
||
|  | ||
|
|
There was a problem hiding this comment.
| {% endcomment %} | |
There was a problem hiding this comment.
Awesome! @forstisabella ! 🙌 I've added that section back in and wrapped it in comment syntax. :) Thanks for the help!
There was a problem hiding this comment.
Awesome @spencerattick!! Glad to hear it - I'll get this merged today and it'll go out with tomorrow's deploy :)
|
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
Proposed changes
Insert Function error information is not surfaced in the Event Delivery view contrary to what was originally published in our docs. A Functions engineer confirmed this: https://twilio.slack.com/archives/CH44TB529/p1689777484339219. Some customers have already expressed confusion about this so I'm removing the reference to Event Delivery until that functionality exists.
Merge timing
ASAP is fine