Skip to content

Update trigger UI labels: Webhooks to Webhook, Alert options to Email#5057

Open
shannonbradshaw wants to merge 1 commit into
mainfrom
claude/update-trigger-ui-labels-2026-05-14
Open

Update trigger UI labels: Webhooks to Webhook, Alert options to Email#5057
shannonbradshaw wants to merge 1 commit into
mainfrom
claude/update-trigger-ui-labels-2026-05-14

Conversation

@shannonbradshaw
Copy link
Copy Markdown
Collaborator

Source changes

  • viamrobotics/app#11852: trigger card UI restructure renamed section headings and button text

Docs changes

  • docs/vision/object-detection/alert-on-detections.md: "ALERT OPTIONS and WEBHOOKS sections" → "EMAIL and WEBHOOK sections"; "click Add Webhook" → "click Add webhook"
  • docs/data/trigger-on-data.md: "Webhooks and Email" → "Webhook and Email" (singular); "Click Add Webhook" → "Click Add webhook"
  • docs/monitor/alert.md: "click Add Webhook" → "click Add webhook"

How I found these

  • Grep matches: searched entire docs repo for ALERT OPTIONS, WEBHOOKS (as section heading), and Add Webhook; found 3 pages with stale UI labels

Generated by daily docs change agent


Generated by Claude Code

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label May 14, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 5b7054e
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/6a186e3bb1aa370008d51d13
😎 Deploy Preview https://deploy-preview-5057--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 41 (🔴 down 2 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

Auto-comment from overwatch. Will not re-nudge for 7 days.

1 similar comment
@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

Auto-comment from overwatch. Will not re-nudge for 7 days.

{{<imgproc src="/tutorials/helmet/trigger.png" resize="x600" style="width: 500px" declaredimensions=true alt="The trigger configured with 'Data has been synced to the cloud' as the type and 'Binary (image)' as the data type." class="shadow imgzoom" >}}

5. Add notification methods. The following options live in the **ALERT OPTIONS** and **WEBHOOKS** sections of the trigger card:
5. Add notification methods. The following options live in the **EMAIL** and **WEBHOOK** sections of the trigger card:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should probably be push notification instead of web hook if I'm understanding correctly

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or maybe it's all 3 actually?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch — it's all three. The trigger card now has EMAIL, WEBHOOK, and PUSH NOTIFICATIONS sections (push went GA in viamrobotics/app#12147, picked up by docs PR #5079). Updated the step to list all three sections and added a Push notifications paragraph parallel to the Webhook one. Also rebased onto main to resolve the conflict from #5079 and removed the redundant trigger.png screenshot at Shannon's request.

@shannonbradshaw shannonbradshaw force-pushed the claude/update-trigger-ui-labels-2026-05-14 branch from 236d45e to f96d0ff Compare May 28, 2026 16:21
The trigger card UI renamed section headings: "Webhooks" is now
"Webhook" (singular), "Alert options" is now "Email", and the
"Add Webhook" button is now "Add webhook" (lowercase). Update three
docs pages that reference these UI labels.

https://claude.ai/code/session_01VHGqDtUDo7U3psP6p9dsi1
@shannonbradshaw shannonbradshaw force-pushed the claude/update-trigger-ui-labels-2026-05-14 branch from f96d0ff to 5b7054e Compare May 28, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants