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

Offline Support: Post / Page Published as Draft #11420

Closed
diegoreymendez opened this issue Apr 8, 2019 · 5 comments
Closed

Offline Support: Post / Page Published as Draft #11420

diegoreymendez opened this issue Apr 8, 2019 · 5 comments

Comments

@diegoreymendez
Copy link
Contributor

diegoreymendez commented Apr 8, 2019

Description:

While creating a new post / page offline and publishing it, the post / page is queued but shows up as a draft. After successfully uploading it, it still stays as a draft.

Steps To Reproduce

  1. Go offline
  2. Create a Post
  3. Click on Publish button in the Editor and publish
  4. When on the Post List, go online.
  5. Click on the post's Retry button.

Decision

As the user intended to publish the post, retrying should also publish the post
To clarify, we should amend the error state to mention posting. Error message should say "Could not upload media. Post not published." Then its clear that retry relates to posting, not just media.
To make this work we need to differentiate the error state in publishing/saving/scheduling scenarios.

  1. User publishes and media upload error (use case in this issue)
    Error Message: "Could not upload media. Post not published."
    Action and behavior: Retry. Retrying publishes the post if media uploads successfully

  2. User schedules and media upload error (scheduling rather than publishing)
    Error Message: "Could not upload media. Post not scheduled."
    Action and behavior: Retry. Retrying schedules the post if media uploads successfully

  3. User saves and media upload error (user did not intend to publish)
    Error Message: "Could not upload media."
    Action and behavior: Retry. Retrying only retries uploading the media as the user has not changed the post state

Related Issues

wordpress-mobile/WordPress-Android#9933

@diegoreymendez diegoreymendez changed the title Offline: Post Published as Draft Offline Support: Post Published as Draft Apr 8, 2019
@diegoreymendez diegoreymendez moved this from To Discuss to Backlog in Offline Support: Posting [iOS] Apr 26, 2019
@diegoreymendez diegoreymendez changed the title Offline Support: Post Published as Draft Offline Support: Post / Page Published as Draft Apr 26, 2019
@diegoreymendez diegoreymendez moved this from Next Sprint to Backlog in Offline Support: Posting [iOS] Apr 26, 2019
@shiki
Copy link
Member

shiki commented May 28, 2019

@wordpress-mobile/ravenclaw @megsfulton @osullivanchris This is related to what @maxme and I agreed on wordpress-mobile/WordPress-Android#9933.

Restating it here for discussion.

  • Android: Publishing while offline and retrying will lead to the post being uploaded and published
  • iOS: Publishing while offline and retrying will lead to the post being uploaded as a draft

The Questions

After a user published a post in the Editor but it failed because of network connectivity...

  1. When the user taps on Retry in the Post List, should the post be published?
  2. When the app automatically uploads the post in the background, should the post be published?

@shiki shiki added the [Status] Needs Discussion The issue needs to be discussed by the team before it can be resolved. label May 28, 2019
@diegoreymendez diegoreymendez added 3 3 point estimation 2 2 point estimation and removed 3 3 point estimation labels Jun 5, 2019
@shiki shiki added the Design Needed A design solution is needed. label Jun 11, 2019
@yaelirub yaelirub moved this from Backlog to In Discovery in Offline Support: Posting [iOS] Jun 22, 2019
@osullivanchris
Copy link

  1. Yes I think the user's intention is to publish the page, so retrying should complete that action. I'd follow the Android behaviour here and publish when retrying. If we're unsure - we could show the confirmation dialogue that we usually do before publishing. But as long as the messaging is clear I think its fine.
  2. So just to confirm I understand...In this case the user has tapped publish, did not have connectivity. Connectivity returns, so we upload in the background? I think this could be a slightly grey area. Is it possible this could happen some time later, and the user might have another version on a different device, for example? Perhaps we could change the status to 'ready to publish' and show a user a toast asking if they want to publish it now. What do you think?

@osullivanchris
Copy link

Discussion has progressed here on Android, will capture the proposal back in this ticket when we have finished resolving. Should be the same on both platforms

@osullivanchris osullivanchris moved this from In Discovery to Ready For Dev in Offline Support: Posting [iOS] Jul 3, 2019
@osullivanchris
Copy link

osullivanchris commented Jul 8, 2019

wordpress-mobile/WordPress-Android#9933

Linked the wrong Android issue previously. We decided the following:

  • As the user intended to publish the post, retrying should also publish the post
  • To clarify, we should amend the error state to mention posting. Error message should say "Could not upload media. Post not published." Then its clear that retry relates to posting, not just media.

To make this work we need to differentiate the error state in publishing/saving/scheduling scenarios. Hopefully not a huge chunk of work, was quick for @maxme on Android. But writing it all down here for clarity.

1. User publishes and media upload error (use case in this issue)
Error Message: "Could not upload media. Post not published."
Action and behaviour: Retry. Retrying publishes the post if media uploads successfully

2. User schedules and media upload error (scheduling rather than publishing)
Error Message: "Could not upload media. Post not scheduled."
Action and behaviour: Retry. Retrying schedules the post if media uploads successfully

3. User saves and media upload error (user did not intend to publish)
Error Message: "Could not upload media."
Action and behaviour: Retry. Retrying only retries uploading the media as the user has not changed the post state

@yaelirub yaelirub removed Design Needed A design solution is needed. [Status] Needs Discussion The issue needs to be discussed by the team before it can be resolved. labels Jul 9, 2019
@diegoreymendez diegoreymendez self-assigned this Jul 17, 2019
@shiki
Copy link
Member

shiki commented Aug 6, 2019

Closing this in favor of #12240 which should sufficiently cover this issue.

@shiki shiki closed this as completed Aug 6, 2019
Offline Support: Posting [iOS] automation moved this from In Review to Done Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

5 participants