Skip to content

Conversation

martinmitrevski
Copy link
Contributor

🔗 Issue Links

Resolves https://linear.app/stream/issue/IOS-788/custom-attachmentsgallery-viewer.

🎯 Goal

Describe why we are making this change.

📝 Summary

Provide bullet points with the most important changes in the codebase.

🛠 Implementation

Provide a detailed description of the implementation and explain your decisions if you find them relevant.

🎨 Showcase

Add relevant screenshots and/or videos/gifs to easily see what this PR changes, if applicable.

Before After
img img

🧪 Manual Testing Notes

Explain how this change can be tested manually, if applicable.

☑️ Contributor Checklist

  • I have signed the Stream CLA (required)
  • This change should be manually QAed
  • Changelog is updated with client-facing changes
  • Changelog is updated with new localization keys
  • New code is covered by unit tests
  • Documentation has been updated in the docs-content repo

@martinmitrevski martinmitrevski requested a review from a team as a code owner April 16, 2025 14:58
@Stream-SDK-Bot
Copy link
Collaborator

Stream-SDK-Bot commented Apr 16, 2025

SDK Size

title develop branch diff status
StreamChatSwiftUI 8.23 MB 8.29 MB +65 KB 🟢

Copy link
Member

@nuno-vieira nuno-vieira left a comment

Choose a reason for hiding this comment

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

Overall looks good, just added some comments to see if we can improve the "future-proofness" of the API


public func makeGalleryView(
mediaAttachments: [MediaAttachment],
author: ChatUser,
Copy link
Member

Choose a reason for hiding this comment

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

Like mentioned in the previous comment maybe better to expose the whole message object?

Copy link
Member

@nuno-vieira nuno-vieira left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

@martinmitrevski martinmitrevski enabled auto-merge (squash) April 21, 2025 21:29
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
75.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@martinmitrevski martinmitrevski merged commit ad6fd9a into develop Apr 21, 2025
11 of 12 checks passed
@martinmitrevski martinmitrevski deleted the gallery-factory-method branch April 21, 2025 22:19
@Stream-SDK-Bot Stream-SDK-Bot mentioned this pull request Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants