Tags

None

Referenced Files

	F6174313: D14473.diff
	Wed, Apr 23, 3:20 AM

	Unknown Object (File)
	Thu, Apr 10, 7:27 AM

	Unknown Object (File)
	Thu, Apr 10, 4:06 AM

	Unknown Object (File)
	Wed, Apr 9, 12:02 PM

	Unknown Object (File)
	Wed, Apr 9, 11:31 AM

	Unknown Object (File)
	Wed, Apr 9, 11:26 AM

	Unknown Object (File)
	Sat, Apr 5, 3:48 PM

	Unknown Object (File)
	Sat, Apr 5, 11:42 AM

View All 29 Files

Subscribers

ashoat

tomek

Details

Reviewers

bartek

Commits

rCOMMa6bdf5a306c7: [ios] Implement hasMultipleFrames() function on iOS

Summary

https://linear.app/comm/issue/ENG-10314/migrate-comm-app-to-use-new-media-module

Depends on D14472

Test Plan

Try to send a gif with animation and without animation and verify that the function returns correct result.
Set mediaType to 'photo_or_video' for gifs in order to trigger the check.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

angelika created this revision.Mar 20 2025, 4:39 AM

angelika held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 20 2025, 4:39 AM

angelika added a child revision: D14474: [native] Use hasMultipleFrames() from MediaModule instead of ffmpeg.Mar 20 2025, 4:40 AM

Harbormaster failed remote builds in B33780: Diff 47484!Mar 20 2025, 4:59 AM

Harbormaster completed remote builds in B33780: Diff 47484.Mar 20 2025, 5:33 AM

angelika published this revision for review.Mar 20 2025, 5:56 AM

bartek accepted this revision.Mar 21 2025, 1:52 AM

This revision is now accepted and ready to land.Mar 21 2025, 1:52 AM

Rebase

Harbormaster completed remote builds in B33831: Diff 47549.Tue, Mar 25, 10:23 PM

Closed by commit rCOMMa6bdf5a306c7: [ios] Implement hasMultipleFrames() function on iOS (authored by angelika). · Explain WhyThu, Mar 27, 12:21 PM

This revision was automatically updated to reflect the committed changes.

angelika added a commit: rCOMMa6bdf5a306c7: [ios] Implement hasMultipleFrames() function on iOS.

[ios] Implement hasMultipleFrames() function on iOS
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 47599

native/expo-modules/comm-expo-package/ios/MediaModule.swift

[ios] Implement hasMultipleFrames() function on iOSClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 47599

native/expo-modules/comm-expo-package/ios/MediaModule.swift

[ios] Implement hasMultipleFrames() function on iOS
ClosedPublic
Actions

Revision Contents
Changeset List