Add Index trigger gesture to webhook headers#298
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3af4716741
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
3af4716 to
fc6aa34
Compare
|
@sarfata I noticed this PR after PRing a related one here. I think adding a |
|
@derek-knox I am selecting the option to send Both trigger to the webhook. |
Ah, makes sense. I removed that part in my PR. Your header vs. my different endpoints to get to the similar reliable routing—difference is single vs. multiple endpoints. |
|
@crc-32 good one to merge |
crc-32
left a comment
There was a problem hiding this comment.
looks good! thanks for the contribution!


Summary
X-Index-Triggersingle-click-holdanddouble-click-holdWhy
Webhook receivers currently get the recording and transcription but cannot tell which configured Index action triggered the request. Exposing the gesture lets integrations route single-click-and-hold and double-click-and-hold recordings to different workflows.
The app already computes this distinction in
RecordingOperationFactory; this change carries it through the upload operation to the HTTP request.Testing
IndexWebhookApi.uploadIfEnabledimplementation and call sitegit diff --checkpasses