Skip to content

docs: document email approvals via webhook relay#30

Open
xzlknr wants to merge 1 commit into
gethelio:mainfrom
xzlknr:docs/email-approval-webhook-relay
Open

docs: document email approvals via webhook relay#30
xzlknr wants to merge 1 commit into
gethelio:mainfrom
xzlknr:docs/email-approval-webhook-relay

Conversation

@xzlknr

@xzlknr xzlknr commented Jun 8, 2026

Copy link
Copy Markdown

Summary

  • document email approvals as a webhook relay recipe instead of a native channel
  • show the approve/deny callback endpoints used by the relay
  • note that API secrets should stay server-side in the relay, not in email URLs

Validation

  • Re-checked issue Docs: email approvals via the webhook relay #17 acceptance criteria against the final diff
  • git diff --check
  • pnpm exec prettier --check docs/approvals.md
  • pnpm --filter @gethelio/proxy build
  • pnpm --filter @gethelio/proxy exec vitest run src/approval/api.test.ts (36 passed)

Note: a full proxy test run surfaced existing Windows/local-environment failures unrelated to this docs change (dist/cli.js missing before build plus temp SQLite EBUSY cleanup), so I ran the relevant approval callback test after building the proxy.

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.

1 participant