docs(android): enhance multimedia attachment

[hritika-cometchat]

Description

Related Issue(s)

Type of Change

• Documentation correction/update
• New documentation
• Improvement to existing documentation
• Typo fix
• Other (please specify)

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

Additional Information

Screenshots (if applicable)

[mintlify]

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

Project	Status	Preview	Updated (UTC)
cometchat	🟢 Ready	View Preview	Jul 15, 2026, 2:02 PM

[mintlify]

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

Project	Status	Preview	Updated (UTC)
cometchat	🟡 Building	–	Jul 15, 2026, 1:57 PM

[jitvarpatil]

Docs review — ✅ Approve (one small nit)

Android multi-attachment docs — the Android counterpart of #428 (iOS), with a new 578-line upload-files.mdx. I verified the entire API against the matching SDK branch (origin/ENG-36734) and it's precise.

✅ API — verified against origin/ENG-36734

• CometChat.createUploadFileRequest(@NonNull String receiverId, …) — CometChat.java:2303
• CometChat.getMaxAttachmentCount(): int (:2317) and getMaxFileSize(): long (:2329)
• UploadFileRequest (:64) with addUploadListener(UploadFileListener) (:245) and removeUploadListener() — no-arg (:250). The doc uses the no-arg form correctly and accurately documents the single-global-slot semantics ("a later addUploadListener() replaces the previous; removeUploadListener() clears it").
• uploadAttachments(List<UploadFileItem>, UploadFileListener) plus the Context overload — the doc covers both and correctly explains why Uri-backed items need Context (to resolve name/size/mime type).
• MediaMessage.getAttachments() / setAttachments(List<Attachment>) — MediaMessage.java:239/243
• UploadResult + onComplete(UploadResult result) — matches the SDK's own javadoc example verbatim.
• Platform correctness: uses Android's createUploadFileRequest / getMaxAttachmentCount / UploadFileListener rather than iOS's uploadFiles / getMaxFileCount / MediaCallbackListener. That's a real platform difference and it's handled right — an easy thing to get wrong when porting docs across platforms. 👍

✅ Structure

New sdk/android/v5/upload-files is wired into nav; 0 nav breaks, 0 orphans, redirects +0. (The 2 broken links the analyzer flagged — /sdk/android/calling-setup, /sdk/android/default-calling — are pre-existing, in unchanged files.)

🟡 Nit — unreferenced image

images/multi_attachments_mobile.png is added but not referenced anywhere in the repo (grepped repo-wide). Either it was meant to be embedded in one of the UI Kit pages (message-composer / core-features) and the embed was missed, or it's a leftover — worth adding the embed or dropping the file.

Note

As with #428 (iOS), the SDK feature lives on a feature branch (ENG-36734) — please confirm the SDK version with createUploadFileRequest / getMaxAttachmentCount ships with the release these docs target.

High-quality, precisely-sourced PR. Ready to merge. 🚀

_{🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against chat-sdk-android multi-attachment branch).}

[jitvarpatil]

Docs review — ✅ Approve (one small nit)

Android multi-attachment docs — the Android counterpart of #428 (iOS), with a new 578-line upload-files.mdx. I verified the entire API against the matching SDK branch (origin/ENG-36734) and it's precise.

✅ API — verified against origin/ENG-36734

• CometChat.createUploadFileRequest(@NonNull String receiverId, …) — CometChat.java:2303
• CometChat.getMaxAttachmentCount(): int (:2317) and getMaxFileSize(): long (:2329)
• UploadFileRequest (:64) with addUploadListener(UploadFileListener) (:245) and removeUploadListener() — no-arg (:250). The doc uses the no-arg form correctly and accurately documents the single-global-slot semantics ("a later addUploadListener() replaces the previous; removeUploadListener() clears it").
• uploadAttachments(List<UploadFileItem>, UploadFileListener) plus the Context overload — the doc covers both and correctly explains why Uri-backed items need Context (to resolve name/size/mime type).
• MediaMessage.getAttachments() / setAttachments(List<Attachment>) — MediaMessage.java:239/243
• UploadResult + onComplete(UploadResult result) — matches the SDK's own javadoc example verbatim.
• Platform correctness: uses Android's createUploadFileRequest / getMaxAttachmentCount / UploadFileListener rather than iOS's uploadFiles / getMaxFileCount / MediaCallbackListener. That's a real platform difference and it's handled right — an easy thing to get wrong when porting docs across platforms. 👍

✅ Structure

New sdk/android/v5/upload-files is wired into nav; 0 nav breaks, 0 orphans, redirects +0. (The 2 broken links the analyzer flagged — /sdk/android/calling-setup, /sdk/android/default-calling — are pre-existing, in unchanged files.)

🟡 Nit — unreferenced image

images/multi_attachments_mobile.png is added but not referenced anywhere in the repo (grepped repo-wide). Either it was meant to be embedded in one of the UI Kit pages (message-composer / core-features) and the embed was missed, or it's a leftover — worth adding the embed or dropping the file.

Note

As with #428 (iOS), the SDK feature lives on a feature branch (ENG-36734) — please confirm the SDK version with createUploadFileRequest / getMaxAttachmentCount ships with the release these docs target.

High-quality, precisely-sourced PR. Ready to merge. 🚀

_{🤖 Automated docs review (Mintlify link/redirect/content checks + API verification against chat-sdk-android multi-attachment branch).}

[jitvarpatil]

Re-review — ✅ Approve

My earlier approval was auto-dismissed by the new commits, so re-reviewing on head cfe50bf (PR grew 8 → 12 files).

✅ Earlier nit fixed

images/multi_attachments_mobile.png is now referenced in ui-kit/android/v6/core-features.mdx (via the "add multi-attachment overview screenshot" commit). 👍

✅ New content checks out

• The 4 new bubble images (audiobubble, filesbubble, imagesbubble, videosbubble) are all referenced by message-bubble-styling.mdx, with descriptive alt text.
• All image references in the changed Android pages resolve — 0 missing.
• Structure still clean: 0 nav breaks, 0 orphans, 0 broken links, redirects +0.
• upload-files.mdx unchanged (578 lines) — API already verified against ENG-36734 (createUploadFileRequest, getMaxAttachmentCount/getMaxFileSize, UploadFileRequest + no-arg removeUploadListener(), both uploadAttachments overloads, MediaMessage.setAttachments, UploadResult).

🟡 One thing to confirm — this PR overwrites a shared image

images/multi_attachments_mobile.png already exists in main and is referenced there by ui-kit/react-native/core-features.mdx. The commit "update multi-attachment overview image for mobile display" replaces that file's content — so the React Native page's screenshot changes too when this merges.

Please confirm the updated image is still appropriate for the RN page. If it's now Android-specific, give it an Android-specific filename instead so the two platforms don't share one asset.

(This also corrects a note I left on #432: since the file pre-exists in main, this is an overwrite of a shared asset — not a duplicate binary add between the two PRs.)

Everything else verifies. Ready to merge. 🚀

_{🤖 Automated docs review — re-check after new commits.}

[jitvarpatil]

Re-review — ✅ Approve

My earlier approval was auto-dismissed by the new commits, so re-reviewing on head cfe50bf (PR grew 8 → 12 files).

✅ Earlier nit fixed

images/multi_attachments_mobile.png is now referenced in ui-kit/android/v6/core-features.mdx (via the "add multi-attachment overview screenshot" commit). 👍

✅ New content checks out

• The 4 new bubble images (audiobubble, filesbubble, imagesbubble, videosbubble) are all referenced by message-bubble-styling.mdx, with descriptive alt text.
• All image references in the changed Android pages resolve — 0 missing.
• Structure still clean: 0 nav breaks, 0 orphans, 0 broken links, redirects +0.
• upload-files.mdx unchanged (578 lines) — API already verified against ENG-36734 (createUploadFileRequest, getMaxAttachmentCount/getMaxFileSize, UploadFileRequest + no-arg removeUploadListener(), both uploadAttachments overloads, MediaMessage.setAttachments, UploadResult).

🟡 One thing to confirm — this PR overwrites a shared image

images/multi_attachments_mobile.png already exists in main and is referenced there by ui-kit/react-native/core-features.mdx. The commit "update multi-attachment overview image for mobile display" replaces that file's content — so the React Native page's screenshot changes too when this merges.

Please confirm the updated image is still appropriate for the RN page. If it's now Android-specific, give it an Android-specific filename instead so the two platforms don't share one asset.

(This also corrects a note I left on #432: since the file pre-exists in main, this is an overwrite of a shared asset — not a duplicate binary add between the two PRs.)

Everything else verifies. Ready to merge. 🚀

_{🤖 Automated docs review — re-check after new commits.}