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.

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

Editor: Navigating away from editor while media uploads causes problems #4784

Closed
hoverduck opened this issue Nov 15, 2016 · 3 comments
Closed

Comments

@hoverduck
Copy link
Contributor

Steps to reproduce the behavior

  1. Open the editor for a new or existing post or page
  2. Add an image
  3. While it's uploading navigate Back to the previous screen
  4. Navigate to the Posts or Pages list
  5. Publish the post/page without opening it for editing first

Expected behavior

To be unable to publish while uploading, like what happens when attempting to publish from the editor

Actual behavior

I'm able to publish the post, which then has a bad img tag in it pointing to a non-existent image. The media upload never completes, leaving an empty entry in the media library. In the GIF below it shows a constant "Uploading" spinner, but in other cases I've seen it show a "Retry" label...and when I clicked that I then got the infinite spinner and no actual upload.

android-bad-upload-media

This animation shows it moving pretty fast, but the upload problem starts as soon as you leave the editor, no matter how fast you publish it.

Tested on Nexus 6P, Android 7.1.1, wp-alpha-27 (shown above on Android 5.1.1, wp-6.0)
@maxme
Copy link
Contributor

maxme commented Nov 15, 2016

The post shouldn't be publishable at this point. I'm pretty sure this is blocked from the "Publish" button in the Editor but not on the "Publish" button in the post list.

Note: Aztec should handle things differently (media uploads will continue after hitting the "Publish" button).

Related to wordpress-mobile/WordPress-Editor-Android#264 wordpress-mobile/WordPress-Editor-Android#439 wordpress-mobile/WordPress-Editor-Android#369

@daniloercoli
Copy link
Contributor

daniloercoli commented Jan 31, 2017

Aztec is not so far away. Marking this ticket as Aztec-obsolete.

@hypest
Copy link
Contributor

hypest commented Jun 5, 2018

Closing as no longer happening. In recent versions of the app, the posts or pages list respect the media-uploading state and do not offer the "Publish" functionality.

@hypest hypest closed this as completed Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants