Skip to content

docs: mark MCP Events as live on mcp.appwrite.io - #140

Merged
ChiragAgg5k merged 1 commit into
mainfrom
docs/events-live
Oct 14, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
docs/events-live

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

The events docs still said Events were not supported and off in production, waiting on appwrite/appwrite#14293. That issue is closed and both --events and --events-push are on at mcp.appwrite.io, so this replaces the banner with the current status, drops the "in progress" headings (and their anchors), and updates the passages that described #14293 as pending. The announcement post (appwrite/appwrite#14466) links to docs/events.md.

Docs only, no code changes.

Related: #127

@hansi-codes

hansi-codes Bot commented Oct 14, 2026

Copy link
Copy Markdown

🔵 Tier A · Mergeable after minor fixes

The only finding is a minor documentation contradiction about whether self-hosted HTTP deployments support Events.

Updates the Events docs to describe webhook and push delivery as live on mcp.appwrite.io, removes the in-progress wording, and refreshes links to the flag documentation. It also updates the development guide's link to the renamed push-delivery section.

Verdict New comments Fixed Still open
✅ Approved 1 0 0
Finding Where
🟡 Reconcile the self-hosted claim with the hosted-only statement docs/events.md:4
Fix with agent prompt
### Issue 1
docs/events.md:4
**Reconcile the self-hosted claim with the hosted-only statement**

This now says self-hosted HTTP deployments can enable Events, but a few lines below the page says Events are served only by the hosted server at mcp.appwrite.io. Readers can't tell whether self-hosted HTTP is supported; please reconcile the two statements.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
📂 Walkthrough · 3
File Change
docs/development.md Updates the link to the push-delivery flag section.
docs/events.md Replaces the unsupported-feature banner and updates the status and Appwrite-size-limit passages.
docs/flags.md Removes in-progress labels and documents the production flag status.

Reviewed ca63061 · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Tier A · Looks good to merge. Summary

Comment thread docs/events.md
> [appwrite/appwrite#14293](https://github.com/appwrite/appwrite/issues/14293)
> lets a webhook's `authPassword` hold the sealed subscription.
> **Live on [mcp.appwrite.io](https://mcp.appwrite.io/)**, with webhook and
> push delivery. Self-hosted HTTP deployments turn them on with the `--events`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reconcile the self-hosted claim with the hosted-only statement

This now says self-hosted HTTP deployments can enable Events, but a few lines below the page says Events are served only by the hosted server at mcp.appwrite.io. Readers can't tell whether self-hosted HTTP is supported; please reconcile the two statements.

Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/events.md
Line: 4

Comment:
**Reconcile the self-hosted claim with the hosted-only statement**

This now says self-hosted HTTP deployments can enable Events, but a few lines below the page says Events are served only by the hosted server at mcp.appwrite.io. Readers can't tell whether self-hosted HTTP is supported; please reconcile the two statements.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

🟡 Minor · documentation · Reply if this doesn't apply.

@ChiragAgg5k
ChiragAgg5k merged commit d64cfbc into main Oct 14, 2026
6 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the docs/events-live branch October 14, 2026 10:22
@ChiragAgg5k ChiragAgg5k self-assigned this Oct 15, 2026
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