Skip to content

docs: remove broken prisma-fullstack example link#2289

Open
Sumit-Mayani wants to merge 1 commit intoexpressjs:redesignfrom
Sumit-Mayani:fix/broken-prisma-fullstack-link
Open

docs: remove broken prisma-fullstack example link#2289
Sumit-Mayani wants to merge 1 commit intoexpressjs:redesignfrom
Sumit-Mayani:fix/broken-prisma-fullstack-link

Conversation

@Sumit-Mayani
Copy link
Copy Markdown

Fixes #2270

The prisma-fullstack link points to prisma/prisma-examples/tree/latest/pulse/fullstack-simple-chat which returns a 404 — that path no longer exists in the prisma-examples repo.

Removed the broken entry from all 10 locale pages (en, de, es, fr, it, ja, ko, pt-br, zh-cn, zh-tw). The prisma-rest-api-ts entry below it remains as it points to a valid URL.

@Sumit-Mayani Sumit-Mayani requested review from a team as code owners April 21, 2026 06:45
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit eb183f3
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/69e8992905210d000852dce2
😎 Deploy Preview https://deploy-preview-2289--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 97 (🟢 up 1 from production)
Accessibility: 100 (🟢 up 13 from production)
Best Practices: 100 (no change from production)
SEO: 100 (🟢 up 6 from production)
PWA: 80 (🟢 up 50 from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@krzysdz
Copy link
Copy Markdown
Contributor

krzysdz commented Apr 21, 2026

Please change target branch to redesign and rebase.

I'm curious why bots set the target branch to main instead of the default one (there already was one before you). Since it's clear that no human has read it (anyone looking at the PR would see that something is wrong), please acknowledge and follow the OpenJS Foundation AI Assistant Policy

This applies also to #2290 and #2291.

@Sumit-Mayani Sumit-Mayani changed the base branch from main to redesign April 22, 2026 09:33
@Sumit-Mayani
Copy link
Copy Markdown
Author

Thanks for the review, @krzysdz. I've now changed the base branch to redesign.

Regarding the OpenJS Foundation AI Assistant Policy: I used an AI tool to help create this PR. I acknowledge and will follow the policy. I have personally reviewed the changes and verified that the broken link (prisma/prisma-examples pulse/fullstack-simple-chat) returns a 404 and no longer exists in the repo.

The same acknowledgment applies to #2290 and #2291.

@Sumit-Mayani Sumit-Mayani force-pushed the fix/broken-prisma-fullstack-link branch from db0e569 to 12e57d2 Compare April 22, 2026 09:42
The prisma/prisma-examples pulse/fullstack-simple-chat path no longer
exists (returns 404). Removed from all locale pages.

Fixes expressjs#2270

Assisted-by AI: Claude (Anthropic)
@Sumit-Mayani Sumit-Mayani force-pushed the fix/broken-prisma-fullstack-link branch from 12e57d2 to eb183f3 Compare April 22, 2026 09:47
@krzysdz krzysdz added docs Issues/pr concerning content 5.x Docs for 5.x version 4.x Docs for 4.x version labels Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.x Docs for 4.x version 5.x Docs for 5.x version docs Issues/pr concerning content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants