Updated code#399
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Aligns the Calls page install instructions with getting-started and upgrading-from-v5, which moved off Cloudsmith to pub.dev in this PR. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Docs review — 🔴 Request changesFlutter docs PR (5 files) doing three good things — documenting the SDK 🔴 P1 — broken code examples (wrong API names; won't compile)1.
(The 2.
✅ What passed (checked)
Both blockers are tightly scoped — happy to push the exact corrected code blocks (right class names, 🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against published cometchat_sdk / cometchat_chat_uikit 6.0.4 and source). |
jitvarpatil
left a comment
There was a problem hiding this comment.
Re-review — ✅ Approve
The two P1 blockers from the earlier review are fixed correctly, verified against the published cometchat_sdk / cometchat_chat_uikit 6.0.4 dartdoc and source.
1. SDK AIAssistantListener example (sdk/flutter/real-time-listeners.mdx) — all event branches now use real classes:
AIAssistantToolStartedEvent(wasAIAssistantToolCallStartedEvent)- the nonexistent
AIAssistantTextMessageStartedEventbranch removed AIAssistantContentReceivedEvent+event.delta(wasAIAssistantTextMessageContentEvent+event.content)AIAssistantMessageEndedEvent(wasAIAssistantTextMessageEndedEvent)RunStarted/Card*/RunFinished+event.id/event.cardId/event.getCard()all correct.
2. UIKit card-action handler (ui-kit/flutter/methods.mdx) — now void ccCardActionClicked(BaseMessage message, dynamic action) with an if (action is CometChatCardActionEvent) check, accessing the real fields elementId, action, cardJson. The invalid param type and nonexistent actionUrl are gone.
Still passing: 0 nav breaks, 0 orphans, 0 missing redirects, 0 broken links in changed files; pub.dev cometchat_chat_uikit: ^6.0.4 migration consistent across all 3 pages; all other SDK/UIKit symbols verified.
Only leftover is a pre-existing broken link /sdk/flutter/calling-overview in sdk/flutter/overview.mdx — unrelated to this PR, worth a separate fix.
LGTM. 🚀
🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against published cometchat_sdk / cometchat_chat_uikit 6.0.4 and source).
|
Docs PR opened: #401 Replaced generic titles on four Flutter SDK and UI Kit pages with descriptive SEO-friendly titles, preserving original labels as sidebarTitle. Note: failed to automatically merge |
Description
Related Issue(s)
Type of Change
Checklist
Additional Information
Screenshots (if applicable)