docs: drop the discontinued extension marketplace - #874
Open
adityarao3 wants to merge 1 commit into
Open
Conversation
The extension system was removed from the app in 0ce07b2 ("Remove the extension system completely"), which replaced the manager with a placeholder reading "Extensions are no longer available". The docs were not updated alongside it, so they still describe a feature that no longer ships and point at infrastructure that is down. marketplace.recordly.dev currently returns HTTP 525 (Cloudflare SSL handshake failed) on both /extensions and /api/extensions, while recordly.dev itself resolves normally. Users hit the raw Cloudflare error page rendered inside the Browse tab, reported in webadderallorg#832 by three people. That request no longer exists in the app, but the README still sends readers to the same dead URL. Changes: - README.md / README.zh-CN.md: remove the "Extensions & Marketplace" section and its link to the marketplace - CONTRIBUTING.md: drop extensions from the list of areas where help is valuable, since contributions there cannot be shipped - EXTENSIONS.md: delete. It documents an API that no longer exists, an `extension-examples/` directory that was removed, an `Extensions -> Open Directory` UI that no longer exists, and links to the same 525 URL. The CONTRIBUTING.md entry above was its only remaining inbound link. docs/announcements.md is left as is: the `extensions` editor section is still a valid announcement target, since the sidebar entry remains as a placeholder. Docs only; no code changes.
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (4)
💤 Files with no reviewable changes (4)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe pull request removes the extension documentation file and removes extension-related references from the contribution guide and English and Chinese README files. ChangesExtension documentation cleanup
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This removes obsolete extension and marketplace documentation while retaining the valid announcement reference. No current merge-readiness risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The extension system was removed from the app in 0ce07b2 ("Remove the extension system completely"), which replaced
ExtensionManagerwith a placeholder reading "Extensions are no longer available". The docs were not updated alongside it, so they still describe a feature that no longer ships and link to infrastructure that is down.marketplace.recordly.devcurrently returns HTTP 525 (Cloudflare SSL handshake failed):https://marketplace.recordly.dev/extensionshttps://marketplace.recordly.dev/api/extensionshttps://recordly.devThis is what #832 reports — the Browse tab rendered the raw Cloudflare error page inline (
Marketplace API error 525: <!DOCTYPE html> ... 525: SSL handshake failed), reproduced by three users. That request no longer exists in the app after 0ce07b2, but the README still sends readers to the same dead URL.To be clear about scope: this PR does not fix the 525. That is a server-side certificate problem on the marketplace origin and only the maintainers can resolve it. This only stops the repo from advertising a removed feature and a dead link.
Changes
Docs only — no code changes.
extension-examples/directory that was removed, anExtensions -> Open DirectoryUI that no longer exists, and links to the same 525 URL. The CONTRIBUTING.md entry above was its only remaining inbound link.docs/announcements.mdis deliberately left unchanged: theextensionseditor section is still a valid announcement target, since the sidebar entry remains as a placeholder.Note for maintainers
If the extension system is coming back, say so and I will rework this to mark the docs as temporarily unavailable instead of removing them — 0ce07b2 has no commit body, so I could not tell whether the removal is permanent or a response to the outage. Either way #832 is worth a reply, since the reporters have had none and the error they saw is now unreachable for a different reason than they think.
Summary by CodeRabbit