Skip to content

docs(notifications): retire orphan /notifications/overview - #513

Open
shardul-cometchat wants to merge 1 commit into
mainfrom
docs/notifications-overview-redirect
Open

docs(notifications): retire orphan /notifications/overview#513
shardul-cometchat wants to merge 1 commit into
mainfrom
docs/notifications-overview-redirect

Conversation

@shardul-cometchat

Copy link
Copy Markdown
Contributor

notifications/overview.mdx was an orphan — not referenced in docs.json nav (flagged repeatedly in the #468 reviews as a pre-existing issue) and a stale duplicate of the /notifications landing page: a thin page with a placeholder description ("Overview of Overview in CometChat") and legacy-extension links, superseded by the richer notifications.mdx hero.

Rather than surface a second, lower-quality overview in the sidebar, this retires it:

  • Delete notifications/overview.mdx.
  • Redirect /notifications/overview -> /notifications (preserves the URL for any external/SEO inbound links).
  • Repoint the two internal links in fundamentals/notification-extensions.mdx (which called it "the modern Notifications platform") to the canonical /notifications.

No nav change needed — /notifications is already the Notifications tab landing. docs.json validated as parseable; no /notifications/overview links remain in any .mdx.

@mintlify

mintlify Bot commented Sep 9, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
cometchat 🟢 Ready View Preview Sep 9, 2026, 8:58 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

notifications/overview.mdx was an orphan (not in docs.json nav) and a stale duplicate
of the /notifications landing page — a thin page with a placeholder description and
legacy-extension links, superseded by the richer notifications.mdx hero.

Delete it, add a redirect /notifications/overview -> /notifications, and repoint the two
links in fundamentals/notification-extensions.mdx to the canonical landing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants