Skip to content

docs(drive): remove webhook references from the Drive guide#100

Merged
nbkdoesntknowcoding merged 1 commit into
nbkdoesntknowcoding:mainfrom
Jason-jo17:docs/drive-webhook-align
Jul 13, 2026
Merged

docs(drive): remove webhook references from the Drive guide#100
nbkdoesntknowcoding merged 1 commit into
nbkdoesntknowcoding:mainfrom
Jason-jo17:docs/drive-webhook-align

Conversation

@Jason-jo17

Copy link
Copy Markdown
Contributor

What & why

Small docs follow-up to the already-merged Drive guide (#98). The backend PR #96 drops the unfinished push webhook, so this removes the now-inaccurate bits from docs/connect/drive.md: the DRIVE_WEBHOOK_SECRET env row and the Google push-notification / POST /api/drive/webhook mention, and scopes the "what syncs" copy to text -> docs and docx/pdf -> attachments (the v1 supported types). Keeps the merged docs consistent with the shipped code.

(This is the alignment commit that got stranded when #98 was squash-merged before it landed - rebased cleanly onto current main here.)

Linked issue

Follow-up to #98; matches #96. Design: #99.

Test evidence

Docs-only, one file (docs/connect/drive.md). Verified no webhook / DRIVE_WEBHOOK_SECRET / images references remain.

Checklist

  • One logical change; matches the surrounding code style.
  • Docs-only - no build/test surface.
  • Commits are signed off for the DCO (git commit -s).
  • This is a core change - not enterprise-adjacent and not licensing/release machinery.
  • I understand this PR is merged here on GitHub once approved and ships in the next tagged release.

…ents

Follows the nbkdoesntknowcoding#96 rework: removes the DRIVE_WEBHOOK_SECRET env row and the
Google-push-notification/webhook mention (that path was dropped as
unfinished; live push-sync is a follow-up), and scopes the "what syncs"
copy to text -> docs and docx/pdf -> attachments (the v1 supported types).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Jason-jo17 <jason@theboringpeople.in>
@nbkdoesntknowcoding nbkdoesntknowcoding merged commit ba890e4 into nbkdoesntknowcoding:main Jul 13, 2026
2 checks passed
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.

2 participants