Skip to content
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.

Sign up for GitHub

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

Improve display of error returned from the REST API when an unsupported image type is uploaded #68788

Merged
merged 13 commits into from
Feb 17, 2025
Merged

Improve display of error returned from the REST API when an unsupported image type is uploaded #68788

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
714fa5a
try: add error when uploading image type not supported by server
adamsilverstein Jan 10, 2025
a1d3fc1
Merge branch 'trunk' of github.com:WordPress/gutenberg into trunk
adamsilverstein Jan 13, 2025
605bf32
Fix settings to match file.type
adamsilverstein Jan 13, 2025
571d4ef
Code cleanup
adamsilverstein Jan 13, 2025
b5c1bd9
Merge branch 'trunk' into fix/image-upload-types
adamsilverstein Jan 20, 2025
f97809b
move new settings to lib/compat
adamsilverstein Jan 20, 2025
2d04445
Try: move mime type support checks to rest api response
adamsilverstein Jan 20, 2025
7c6a647
Merge branch 'trunk' into fix/image-upload-types-via-rest-api
adamsilverstein Feb 10, 2025
a027439
reintroduce after merge, linting
adamsilverstein Feb 10, 2025
44cdcca
remove inadvertant changes
adamsilverstein Feb 10, 2025
f993d76
remove inadvertant changes
adamsilverstein Feb 10, 2025
9817117
Simplify logic and apply to all image types
adamsilverstein Feb 10, 2025
db2f541
revert rest api part
adamsilverstein Feb 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 10 additions & 3 deletions packages/media-utils/src/utils/upload-media.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,9 +134,16 @@ export function uploadMedia( {
// Reset to empty on failure.
setAndUpdateFiles( index, null );

let message;
if ( error instanceof Error ) {
message = error.message;
let message: string;
adamsilverstein marked this conversation as resolved.
Show resolved Hide resolved
if (
typeof error === 'object' &&
error !== null &&
'message' in error
) {
message =
typeof error.message === 'string'
? error.message
: String( error.message );
} else {
message = sprintf(
// translators: %s: file name
Expand Down
Loading