Present a generic error to the user when plugin is unable to auto-import into site Media Library a

[bmattb]

Feature description

Occasionally media upload will fail, and we are unable to build out specific responses for each case with the limited time left. In cases such as this we should present a generic error to the user that recommends a retry.

I would expect that if an import action should fail that the plugin will present the following message to users via an in-browser notification. The notification should state:

"The file was unable to be imported into the Media Library. Please try again"

and present the user with the "Ok" button to acknowledge and clear the pop-up

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

• user is presented with a generic error message when there is a failure to import selected Unsplash images into post content and/or the site's Media Library for unidentified reasons.

Implementation brief

• In the ImportImages method invocation catch statement (

  unsplash-wp/assets/src/media-selector/views/button.js

  Line 43 in a63e5bb

  // TODO: let user know import failed

  ), display a browser alert with the message defined above.

QA testing instructions

• Create a new post.
• Add an image block.
• Open media selector.
• Select Unsplash tab.
• Select an image.
• Open new tab.
• Go to Unsplash settings page. Deauthicate the plugin.
• Return to original tab. With image still selected, try to insert to post.
• Error message should be displayed.

Demo

Changelog entry

[csossi]

@ravichdev - any details on how I can test this one? what conditions need to be present?

[ravichdev]

@csossi This is difficult to test, something has to cause an image import from Unsplash to fail, on my local I tested this with adding logic to the import code to exit early, but it's difficult for you to test this. @spacedmonkey Should we just move this to Done as you already reviewed the code and good with the solution ?

[spacedmonkey]

@csossi See Testing instructions.

[csossi]

Verified in QA

Error popup appearing:

[bmattb]

@spacedmonkey - looks like the testing instructions didn't trigger the expected warning... thoughts?

[bmattb]

this has been flagged as "untestable" in current environments so I will pass and hope that it's working as expected.