Skip to content

Point Appwrite Network CTA to docs and retire map-only page#2919

Merged
atharvadeosthale merged 1 commit intomainfrom
fix/homepage-appwrite-network-docs-link
Apr 28, 2026
Merged

Point Appwrite Network CTA to docs and retire map-only page#2919
atharvadeosthale merged 1 commit intomainfrom
fix/homepage-appwrite-network-docs-link

Conversation

@eldadfux
Copy link
Copy Markdown
Member

Homepage and network-map components now link to /docs/products/network. Remove /the-appwrite-network standalone page; add 308 redirect to docs.

Made-with: Cursor

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Homepage and network-map components now link to /docs/products/network.
Remove /the-appwrite-network standalone page; add 308 redirect to docs.

Made-with: Cursor
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 28, 2026

Greptile Summary

This PR retires the standalone /the-appwrite-network page (which only rendered a map) and redirects it to /docs/products/network, updating the two marketing component CTAs accordingly. As a bonus, it also fixes a pre-existing bug in network-map.svelte where the "More about the Appwrite Network" link pointed to "/" (the homepage) instead of the intended network destination.

Confidence Score: 5/5

Safe to merge — all changes are clean link updates and a well-handled redirect.

No logic issues found. The redirect uses the existing 308 mechanism in hooks.server.ts, CTAs are correctly updated, the deleted page has no remaining internal references to the standalone route, and a pre-existing broken link is also fixed in the process.

No files require special attention.

Important Files Changed

Filename Overview
src/redirects.json Adds 308 redirect from /the-appwrite-network to /docs/products/network; hooks.server.ts already applies status 308 for all entries in this file.
src/routes/(marketing)/(components)/map.svelte Updates CTA button href from /the-appwrite-network to /docs/products/network; straightforward and correct.
src/routes/(marketing)/(components)/network-map.svelte Fixes a pre-existing bug (href was "/" pointing to homepage) and updates to /docs/products/network.
src/routes/the-appwrite-network/+page.svelte Deleted standalone map-only page; covered by new redirect entry in redirects.json.

Reviews (1): Last reviewed commit: "Point Appwrite Network CTA to docs and r..." | Re-trigger Greptile

@atharvadeosthale atharvadeosthale merged commit 32f1aa3 into main Apr 28, 2026
6 checks passed
@atharvadeosthale atharvadeosthale deleted the fix/homepage-appwrite-network-docs-link branch April 28, 2026 06:35
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