-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Description
This issues is currently reproducible only on issue/10174-update-changes-confirmed-at.
After editing a published post and exiting the editor, the remote auto-save does not seem to get executed right away. I can force the remote auto-save by swiping to refresh. Here is a video showing this. And the steps:
- Edit a published post.
- Make changes.
- Exit the editor by pressing the Back button.
- Confirm in Calypso that no revision was created.
- 00:23 Swipe to refresh. An upload feedback UI will show that the post is being remote auto-saved.
- Confirm in Calypso that a revision was created.
Thank you @shiki for finding and recording the issue;)!
Note: This if statement is causing these troubles. I think we should completely get rid of it and get rid of savePostLocallyAndFinishAsync. As we always want to invoke remote-auto-save (with exception for self-hosted sites). Just be careful that we'll need to handle
WordPress-Android/WordPress/src/main/java/org/wordpress/android/ui/posts/EditPostActivity.java
Line 2133 in 5516721
| PendingDraftsNotificationsUtils.scheduleNextNotifications(EditPostActivity.this, mPost); |