Add article: Static sites vs. SSR on Appwrite Sites#2870
Add article: Static sites vs. SSR on Appwrite Sites#2870
Conversation
Agent-Logs-Url: https://github.com/appwrite/website/sessions/7a646408-6784-41f2-9986-a9892502bb9e Co-authored-by: Meldiron <19310830+Meldiron@users.noreply.github.com>
Greptile SummaryThis PR adds a new blog post, "Static sites vs. SSR on Appwrite Sites: how to choose," which clearly explains the two rendering models, their trade-offs, and when to use each. The article is well-written, well-structured, and all referenced internal blog posts exist. Two content-level issues need attention before publish:
Confidence Score: 3/5Hold for content fixes — two accuracy issues and a missing cover image asset need to be resolved before this is ready to publish. The orphaned table row (a concept without any explanation in the article) and the potentially inaccurate claim about per-route SSR granularity are both P1 content correctness issues that could mislead readers. The missing cover image is also unresolved per the PR description itself. Together these warrant a 3/5 until addressed. src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc — lines 78 and 88 need review; cover image asset still missing.
|
| Filename | Overview |
|---|---|
| src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc | New blog post explaining static vs. SSR on Appwrite Sites — content is well-structured and technically sound overall, but has two issues: the on-demand cache invalidation table row is orphaned (its explanation paragraph was removed), and one sentence may overstate per-route rendering granularity on Appwrite Sites. |
Reviews (4): Last reviewed commit: "Remove on-demand cache invalidation sect..." | Re-trigger Greptile
src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc
Outdated
Show resolved
Hide resolved
src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc
Outdated
Show resolved
Hide resolved
src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc
Outdated
Show resolved
Hide resolved
…to-catch bugs Agent-Logs-Url: https://github.com/appwrite/website/sessions/e6733b1c-543c-46d1-a7b1-4a9b431b8ac2 Co-authored-by: Meldiron <19310830+Meldiron@users.noreply.github.com>
|
Tip: Greploops — Automatically fix all review issues by running Use the Greptile plugin for Claude Code to query reviews, search comments, and manage custom context directly from your terminal. |
src/routes/blog/post/static-vs-ssr-appwrite-sites/+page.markdoc
Outdated
Show resolved
Hide resolved
Agent-Logs-Url: https://github.com/appwrite/website/sessions/9954a253-8d3f-4a2b-a05d-f54a6b0587f2 Co-authored-by: Meldiron <19310830+Meldiron@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/appwrite/website/sessions/cfe6f996-a182-4cb6-8ed5-3d4bf237e2cc Co-authored-by: Meldiron <19310830+Meldiron@users.noreply.github.com>
| | Pre-built documentation or marketing pages | Static | | ||
| | User-generated content at scale with SEO requirements | SSR | | ||
| | Personalized pages rendered per-request | SSR | | ||
| | On-demand cache invalidation for individual pages | SSR | |
There was a problem hiding this comment.
Orphaned table row references removed content
The PR description confirms that the "on-demand cache invalidation" paragraph was removed from the article body (via the [x] Removed on-demand cache invalidation paragraph checklist item). However, line 78 still lists On-demand cache invalidation for individual pages | SSR in the decision table without any explanation elsewhere in the article.
Readers who encounter this row will have no context for what it means or why SSR handles it better than static hosting — the supporting explanation no longer exists. Either re-add a brief explanation in the body, or remove this row from the table to keep the guide consistent with the article's content.
Uh oh!
There was an error while loading. Please reload this page.