Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Exposed factory methods for gallery and video player view #808

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 21, 2025

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

@@ -445,6 +445,32 @@ extension ViewFactory {
)
}

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?

Sources/StreamChatSwiftUI/DefaultViewFactory.swift Outdated Show resolved Hide resolved
Sources/StreamChatSwiftUI/DefaultViewFactory.swift Outdated Show resolved Hide resolved
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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.