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

Double draft post created when saving as draft after video upload failed #12073

Closed
etoledom opened this issue Jul 8, 2019 · 4 comments
Closed

Comments

@etoledom
Copy link
Contributor

etoledom commented Jul 8, 2019

Expected behavior

Just one draft post is created.

Actual behavior

Two draft posts are created

Steps to reproduce the behavior

  • Start a new post (Aztec or Gutenberg)
  • Add a video from the camera roll
  • While it is uploading, turn off network connection to make the upload fail
  • With the network off, save the post as Draft without leaving the post
  • Turn network connection back on - the video will start uploading again
  • When the video finishes uploading, two posts will be created and saved as draft
  • Check on web that two posts were created or go back to the post list and pull to refresh.
Tested on iPhone XS, iOS 12.3.1, WPiOS 12.8 Internal
@shiki
Copy link
Member

shiki commented Jul 8, 2019

This could be related to wordpress-mobile/WordPress-Android#9845 where we found that:

  1. The post could have been uploaded and received by the server. Hence, a post in the server was created.
  2. The client (app) did not receive the server response and assumed the post was not created.

So, the client will try the upload again, creating another post.

@stale
Copy link

stale bot commented Jul 11, 2020

This issue has been marked as stale because:

  • It has been inactive for the past year.
  • It isn't in a project or a milestone.
  • It hasn’t been labeled [Pri] Blocker, [Pri] High, or good first issue.

Please comment with an update if you believe this issue is still valid or if it can be closed. This issue will also be reviewed for validity and priority (cc @designsimply).

@stale stale bot added the [Status] Stale label Jul 11, 2020
@peril-wordpress-mobile
Copy link

Fails
🚫 Please add a feature label to this issue. e.g. 'Stats'

Generated by 🚫 dangerJS

@kean
Copy link
Contributor

kean commented May 4, 2024

This system was re-implemented in 24.9 – closing as obsolete.

@kean kean closed this as completed May 4, 2024
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

3 participants