-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Auto-upload Published] Show Publish button in Post List #12321
Comments
@shiki the proposal you shared with the 'publish' button returning after cancelling looks correct to me and what I would expect. So I'm happy to go with that! |
@osullivanchris I was talking with @yaelirub about this yesterday. It looks like I failed to communicate what I meant for “showing the Publish button“. I meant to communicate that we will follow how Android's Publish button works. Android has been showing the Publish button even before the Cancel button even existed. In Android, the Publish button is shown when any of these conditions are true (ref):
However, Yael understood this ticket as “show the Publish button after canceling only“, so clearly I failed in communicating this well. There's a significant difference in the implementation of both. For example, the former does not require any change in the database while the latter requires a new property (flag) in the database. I'd like to hear what you think. Which one makes sense? Should we follow Android or implement it differently on iOS? 😐 |
That's what I thought it meant :)
That's fine. The error message will also have changed in this scenario right? So it won't be saying its going to publish anymore.
I don't really understand this so I guess its a technical decision?
Is there any alternative to the proposed solution? I can't think of what other way we would implement it :) |
Okay. I suppose we could try and do that then.
The other way is not doing anything at all. 🙂 To reconfirm auto-upload, our users would need to:
The Post List would look empty though: |
Oh, sorry. To clarify, there are three options: a. Show Publish only when Cancel was pressed. But I believe this discussion is leading towards a. Please let me know otherwise. 🙂 |
I wonder if C makes more sense. Not to say that we won't do it but to delay it after Post Logic Improvements (I haven't seen the scope for that yet). With that said, I also realize that we want to align features on both platforms so I'd go with A. Leaving the decision to you, @shiki . |
If I understand correctly, if we go with option A or C there will be some scenarios where the user is shown an error message and no option to publish the post. That feels like we're creating a dead end for the user. I don't really see what the advantage of that option is. So (b) to me sounds like the only viable UX. But I might have misunderstood something, or maybe the cost is huge with (b). @shiki if you want to jump on a quick call to talk this through today it might be quicker to get to the bottom of. |
Done in #12375. |
📣 Part of #12240. To decrease the risk of conflicts, this should use the #12317 PR branch as the base branch. Ideally, that branch should be merged first. Any fixes should target the
issue/12240-master-branch
branch.In #12317, we added the Cancel button in the Post List. When the user presses the Cancel button, the app will not try to automatically upload the post anymore. The app will then hide the Cancel button and only show Edit and More.
There used to be a Retry button in the middle but I removed it since it seems to be confusing if it is shown after cancelation.
In WPAndroid, we show a Publish button after canceling.
Proposal
Questions
@osullivanchris Hoping to get your input here. 🙂
Decisions
Discovered Tasks
Pressing on the Publish button should mark the post as “confirmed for auto-uploading“. If offline, the post will be automatically uploaded later.[Offline Posting] Publish button in Post List should auto-upload the post #12382The text was updated successfully, but these errors were encountered: