Skip to content

Use default content types for document picker#25438

Merged
crazytonyli merged 1 commit intorelease/26.8from
bugfix/empty-allowed-files
Mar 24, 2026
Merged

Use default content types for document picker#25438
crazytonyli merged 1 commit intorelease/26.8from
bugfix/empty-allowed-files

Conversation

@crazytonyli
Copy link
Contributor

Description

Context: p1774261168818399/1772729090.522759-slack-C06NXHR5WQ1

The original code returns an empty array when allowedFileTypes are all invalid for UTType. I can't be sure that's the case for the customer's site, though.

@crazytonyli crazytonyli added this to the 26.8 ❄️ milestone Mar 24, 2026
@crazytonyli crazytonyli requested a review from kean March 24, 2026 05:18
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is assigned to the milestone 26.8 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

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 Number31714
VersionPR #25438
Bundle IDorg.wordpress.alpha
Commit9830151
Installation URL0t2krcvo67138
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

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 Number31714
VersionPR #25438
Bundle IDcom.jetpack.alpha
Commit9830151
Installation URL1vb7dnp76er10
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@kean kean left a comment

Choose a reason for hiding this comment

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

It's not clear if it fixes something, but looks like a safe change to make.

@crazytonyli crazytonyli merged commit 9a1f78a into release/26.8 Mar 24, 2026
29 checks passed
@crazytonyli crazytonyli deleted the bugfix/empty-allowed-files branch March 24, 2026 20:12
github-merge-queue bot pushed a commit that referenced this pull request Mar 24, 2026
* Remove the Post Settings feature flag (#25428)

* Use default content types for document picker (#25438)

* Add "BETA" to the Custom Post Types based post list screens (#25436)

* Add "BETA" to the Custom Post Types based post list screens

* Update wordpress-rs

* Remove application password availablity check (#25439)

* Update strings for localization

* Update app translations – `Localizable.strings`

* Update WordPress metadata translations

* Bump version number

---------

Co-authored-by: Tony Li <tony.li@automattic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants