Added agent in group#419
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
jitvarpatil
left a comment
jitvarpatil
left a comment
There was a problem hiding this comment.
Docs review — 🟠 Request changes (broken link)
Strong PR overall — the interactive-messages deletion is handled correctly with a redirect (→ /sdk/flutter/send-message), version bumps (cometchat_chat_uikit ^6.0.5, cometchat_calls_sdk ^5.0.3) are both published/current, cards are correctly documented as receive-only, and the ccCardActionClicked example uses the right dynamic + CometChatCardActionEvent cast pattern (verified against the 6.0.5 UIKit dartdoc). Holding approval on one broken link.
🟠 Please fix: broken internal link
sdk/flutter/overview.mdx:81 links to a page that doesn't exist:
<Card title="Calling" icon="phone" href="/sdk/flutter/calling-overview">
There is no /sdk/flutter/calling-overview page. The Flutter calling docs live under /calls/flutter/ — update the href to /calls/flutter/overview (which exists).
It pre-dates this PR, but since this is the Flutter SDK docs pass, please fix it here (or confirm the intended destination).
🟡 P2 — nit (optional)
sdk/flutter/messaging-overview.mdx:4 — the page description still says "…action, call, and interactive message structures," but the body swapped interactive → card. Update the description to "card" for consistency (body is already correct).
Everything else is verified and good to merge once the link is fixed.
🤖 Automated docs review (Mintlify link/redirect/content checks + version/API verification against pub.dev 6.0.5 / 5.0.3).
…tion - overview.mdx: point Calling card to /calls/flutter/overview (was dead /sdk/flutter/calling-overview) - messaging-overview.mdx: description "interactive" -> "card" to match body Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Re-review — ✅ ApproveBoth findings from the previous review are fixed:
Full structural check clean: 0 broken links, 0 nav breaks, 0 orphans; the deleted Ready to merge. 🚀 🤖 Automated docs review — re-check after fixes. |
jitvarpatil
left a comment
jitvarpatil
left a comment
There was a problem hiding this comment.
Re-review — ✅ Approve
Both findings from the previous review are fixed:
- Broken link fixed:
sdk/flutter/overview.mdx:81nowhref="/calls/flutter/overview"(target exists). Analyzer: 0 broken links (was 1). - Description fixed:
messaging-overview.mdx:4now reads "…custom, card, action, and call message structures" (dropped the stale "interactive").
Full structural check clean: 0 broken links, 0 nav breaks, 0 orphans; the deleted interactive-messages page is still properly redirected → /sdk/flutter/send-message (0 unredirected). Combined with the earlier content verification — published cometchat_chat_uikit ^6.0.5 / cometchat_calls_sdk ^5.0.3, cards correctly documented as receive-only, and the correct ccCardActionClicked + CometChatCardActionEvent cast pattern (verified against the 6.0.5 UIKit dartdoc) — there are no outstanding issues.
Ready to merge. 🚀
🤖 Automated docs review — re-check after fixes.
4 participants
Description
Related Issue(s)
Type of Change
Checklist
Additional Information
Screenshots (if applicable)