Skip to content

docs: add loadLastAgentConversation prop to CometChatMessageList (React Native)#390

Merged
ketanyekale merged 6 commits into
mainfrom
docs/rn-message-list-load-last-agent-conversation
Jun 26, 2026
Merged

docs: add loadLastAgentConversation prop to CometChatMessageList (React Native)#390
ketanyekale merged 6 commits into
mainfrom
docs/rn-message-list-load-last-agent-conversation

Conversation

@suraj-chauhan-cometchat

Copy link
Copy Markdown
Contributor

Description

Added the loadLastAgentConversation prop to CometChatMessageList documentation. This boolean prop enables automatic loading of the most recent agent conversation when set to true.

Type of Change

  • Documentation correction/update
  • New documentation

Changes Made

  • Added loadLastAgentConversation to the AI Quick Reference JSON (ai props section)
  • Added loadLastAgentConversation prop entry in the Props section (type: boolean, default: false)

Checklist

  • I have read the CONTRIBUTING document
  • My branch name follows the naming convention
  • My changes follow the documentation style guide
  • I have checked for spelling and grammar errors
  • All links in my changes are valid and working
  • My changes are accurately described in this pull request

- Add prop to AI Quick Reference JSON
- Add prop entry in Props section (boolean, default: false)
- Enables automatic loading of the most recent agent conversation
@mintlify

mintlify Bot commented Jun 9, 2026

Copy link
Copy Markdown

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

Project Status Preview Updated (UTC)
cometchat 🟢 Ready View Preview Jun 9, 2026, 4:59 AM

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

…ageList

- Add loadLastAgentConversation prop documentation to props table with type, default, and description
- Remove duplicate loadLastAgentConversation entry from optional props section
- Add new code example demonstrating how to use loadLastAgentConversation to resume recent agent conversations
- Consolidate prop definitions to eliminate redundancy and improve documentation clarity
@raj-dubey1 raj-dubey1 self-requested a review June 24, 2026 11:38
raj-dubey1
raj-dubey1 previously approved these changes Jun 24, 2026
@jitvarpatil

Copy link
Copy Markdown
Contributor

Docs review — ✅ Approve

Single-file additive PR documenting the new loadLastAgentConversation boolean prop on the React Native CometChatMessageList (RN analog of the iOS change in #400). No structural risk; the documented API checks out against source.

✅ What passed (checked)

  • API correctness — verified against origin/master-v5 (released v5 line; npm latest 5.3.8):
    • loadLastAgentConversation?: boolean; (CometChatMessageList.tsx:530) — matches documented type boolean
    • defaulted to false (loadLastAgentConversation = false, :655) — matches documented default false
    • actually consumed in component logic (:937, :2431) — a real functional prop, not a stub
    • usage <CometChatMessageList user={agentUser} loadLastAgentConversation={true} /> and the @cometchat/chat-uikit-react-native import are correct
  • Internal consistency: 3 coherent mentions, no duplicate prop section — AI Quick-Reference JSON (line 64), usage example (line 687), Props entry (line 1060); JSON and prose both agree on boolean/false.
  • House style: Type/Default table and JSON entry shape match sibling props in the file; no placeholders/TODOs.
  • Build / structure: 0 unresolved nav refs, 0 removed/renamed URLs (0 redirects needed).

This change is mergeable as-is.

🔧 Pre-existing issues to fix (NOT introduced by this PR — flagging for a separate cleanup)

Broken internal link/ui-kit/react-native/getting-started does not resolve (no such page). Real slugs are react-native-cli-integration, expo-integration, or guide-overview. Occurrences:

  • ui-kit/react-native/campaigns.mdx:9[UI Kit integration](/ui-kit/react-native/getting-started)
  • ui-kit/react-native/property-changes.mdx:666<Card title="Getting Started" ... href="/ui-kit/react-native/getting-started">

Orphaned pages (exist but absent from docs.json navigation, so unreachable via the sidebar):

  • /ui-kit/react-native/calling-integration
  • /ui-kit/react-native/property-changes

(The -109 redirect delta the analyzer reported is just branch staleness — this branch is behind main and doesn't touch docs.json, so it's a non-issue for merge.)

🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against released cometchat RN UIKit source).

…pages to nav

- Replace /ui-kit/react-native/getting-started (404) with /ui-kit/react-native/overview in campaigns.mdx and property-changes.mdx
- Add calling-integration to Features group in docs.json (before call-features)
- Add property-changes to Migration Guide group in docs.json

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
raj-dubey1
raj-dubey1 previously approved these changes Jun 24, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
raj-dubey1
raj-dubey1 previously approved these changes Jun 24, 2026
MDX does not allow <!-- --> syntax, causing parse failures on deploy.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
jitvarpatil
jitvarpatil previously approved these changes Jun 25, 2026

@jitvarpatil jitvarpatil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-review — ✅ Approve

The original prop is correct and every pre-existing issue from the earlier review has been addressed. The PR grew from 1 file to 4, all improvements.

Resolved

  • Broken getting-started links → fixed: both now point to /ui-kit/react-native/overview (valid) — campaigns.mdx and property-changes.mdx:666. Analyzer confirms 0 broken links (was 2).
  • Orphans → fixed: ui-kit/react-native/calling-integration and …/property-changes added to docs.json nav (lines 1113, 1178). Analyzer confirms 0 orphans (was 2).
  • loadLastAgentConversation prop — intact, all 3 mentions present (JSON quick-ref L65, usage example L689, props section L1062). Verified correct against master-v5 source (loadLastAgentConversation?: boolean default false).

Vale-linter changes — checked, safe

The new {/* vale off */}/{/* vale on */} MDX comments (replacing HTML comments) are valid JSX comments and correctly placed outside the ```json fence (vale-off at message-list.mdx:6 before the fence, vale-on at L122 after it) — they don't break the JSON block or rendering.

Structural

0 nav breaks, 0 orphans, 0 broken links, 0 missing redirects. The -109 redirect delta is just branch staleness (the PR only adds nav entries; removes no redirects) — a rebase on main would reconcile the count, but it's not a blocker.

LGTM. 🚀

🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against cometchat RN UIKit master-v5 source).

@ketanyekale ketanyekale merged commit 007a6dc into main Jun 26, 2026
6 of 7 checks passed
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.

4 participants