Skip to content

Conversation

@crazytonyli
Copy link
Contributor

No description provided.

@crazytonyli crazytonyli added this to the 26.6 milestone Dec 11, 2025
@crazytonyli crazytonyli requested a review from kean December 11, 2025 22:41
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Dec 11, 2025

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number30133
VersionPR #25058
Bundle IDorg.wordpress.alpha
Commitdd90729
Installation URL3h2kmkkbt5lug
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Dec 11, 2025

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number30133
VersionPR #25058
Bundle IDcom.jetpack.alpha
Commitdd90729
Installation URL67r14d58i051o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@crazytonyli crazytonyli changed the title Translate MediaType to Swift Translate a couple of enums in Media to Swift Dec 11, 2025
@sonarqubecloud
Copy link

Comment on lines +13 to +23
switch self {
case .image:
return "image"
case .video:
return "video"
case .powerpoint:
return "powerpoint"
case .document:
return "document"
case .audio:
return "audio"
Copy link
Contributor

Choose a reason for hiding this comment

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

(nit)

Suggested change
switch self {
case .image:
return "image"
case .video:
return "video"
case .powerpoint:
return "powerpoint"
case .document:
return "document"
case .audio:
return "audio"
switch self {
case .image: "image"
case .video: "video"
case .powerpoint: "powerpoint"
case .document: "document"
case .audio: "audio"

@crazytonyli crazytonyli added this pull request to the merge queue Dec 16, 2025
Merged via the queue into trunk with commit c335391 Dec 16, 2025
26 of 32 checks passed
@crazytonyli crazytonyli deleted the core-data-media-enums branch December 16, 2025 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants