KEMBAR78
examples: Remove AMP example by devjiwonchoi · Pull Request #84355 · vercel/next.js · GitHub
Skip to content

Conversation

@devjiwonchoi
Copy link
Member

@devjiwonchoi devjiwonchoi commented Sep 29, 2025

Stacked on #84312

Removes the AMP example, which is no longer supported.

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. examples Issue was opened via the examples template. labels Sep 29, 2025
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Contributor

@vercel vercel bot left a comment

Choose a reason for hiding this comment

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

Additional Comments:

docs/02-pages/02-guides/amp.mdx (line 10):

The AMP example link will become a broken link (404) after the examples/amp directory is deleted. The same issue exists in docs/02-pages/04-api-reference/03-functions/use-amp.mdx at line 9.

View Details
📝 Patch Details
diff --git a/docs/02-pages/02-guides/amp.mdx b/docs/02-pages/02-guides/amp.mdx
index 051a3e123a..4d14b5b1ba 100644
--- a/docs/02-pages/02-guides/amp.mdx
+++ b/docs/02-pages/02-guides/amp.mdx
@@ -4,13 +4,6 @@ nav_title: AMP
 description: With minimal config, and without leaving React, you can start adding AMP and improve the performance and speed of your pages.
 ---
 
-<details>
-  <summary>Examples</summary>
-
-- [AMP](https://github.com/vercel/next.js/tree/canary/examples/amp)
-
-</details>
-
 > **Warning**: Built-in AMP support will be removed in Next.js 16.
 
 With Next.js you can turn any React page into an AMP page, with minimal config, and without leaving React.
diff --git a/docs/02-pages/04-api-reference/03-functions/use-amp.mdx b/docs/02-pages/04-api-reference/03-functions/use-amp.mdx
index 6237a4fb88..e10eca1b16 100644
--- a/docs/02-pages/04-api-reference/03-functions/use-amp.mdx
+++ b/docs/02-pages/04-api-reference/03-functions/use-amp.mdx
@@ -3,13 +3,6 @@ title: useAmp
 description: Enable AMP in a page, and control the way Next.js adds AMP to the page with the AMP config.
 ---
 
-<details>
-  <summary>Examples</summary>
-
-- [AMP](https://github.com/vercel/next.js/tree/canary/examples/amp)
-
-</details>
-
 > **Warning**: Built-in AMP support will be removed in Next.js 16.
 
 To enable AMP, add the following config to your page:

Analysis

Broken links to deleted AMP example in documentation files

What fails: docs/02-pages/02-guides/amp.mdx (line 10) and docs/02-pages/04-api-reference/03-functions/use-amp.mdx (line 9) reference https://github.com/vercel/next.js/tree/canary/examples/amp, which was deleted in commit c03bfc4074

How to reproduce:

# Verify the directory was deleted
git show c03bfc4074 --stat | grep "examples/amp"

# Check broken links in docs
grep "examples/amp" docs/02-pages/02-guides/amp.mdx docs/02-pages/04-api-reference/03-functions/use-amp.mdx

Result: Links return 404 when users click them in the documentation

Expected: Documentation should not contain broken links to deleted examples

@devjiwonchoi devjiwonchoi marked this pull request as ready for review September 29, 2025 21:31
@devjiwonchoi
Copy link
Member Author

docs/02-pages/02-guides/amp.mdx (line 10):

The AMP example link will become a broken link (404) after the examples/amp directory is deleted. The same issue exists in docs/02-pages/04-api-reference/03-functions/use-amp.mdx at line 9.

The docs will be deleted too.

@devjiwonchoi devjiwonchoi merged commit a843a49 into canary Sep 30, 2025
80 checks passed
@devjiwonchoi devjiwonchoi deleted the jiwon/09-29-examples_remove_amp_example branch September 30, 2025 13:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

created-by: Next.js team PRs by the Next.js team. examples Issue was opened via the examples template. locked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants