Grouping image and video attachments in the same message

[martinmitrevski]

🔗 Issue Link

Resolves https://stream-io.atlassian.net/browse/PBE-4343?atlOrigin=eyJpIjoiNWE3NzBlZThiZDUwNGNiM2IzNTRlODBkYmI5ZTFiZDQiLCJwIjoiaiJ9.

🎯 Goal

Images and videos should be part of the same message.

🛠 Implementation

Refactored the existing image attachment view to support videos. Kept the existing ones for backward compatibility.

🧪 Testing

Try different combinations of images and videos (and single ones too).

🎨 Changes

Add relevant screenshots or videos showcasing the changes.

☑️ Checklist

• I have signed the Stream CLA (required)
• Changelog is updated with client-facing changes
• New code is covered by unit tests
• Affected documentation updated (docusaurus, tutorial, CMS (task created)

[laevandus]

I tried the branch and works well.
What I observed was that video attachments are rendered as first items always in-spite of the order how I picked them (pick images first, then videos).

[martinmitrevski]

I tried the branch and works well. What I observed was that video attachments are rendered as first items always in-spite of the order how I picked them (pick images first, then videos).

Yes, that's fine for now. Otherwise we would need to fetch all type erased attachments and cast them, so it might slow things down. We can revisit if there are complaints.

[sonarcloud]

Quality Gate passed

Issues
88 New issues
0 Accepted issues

Measures
0 Security Hotspots
72.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud