-
Notifications
You must be signed in to change notification settings - Fork 120
Closed
Labels
feature: notificationsRelated to notifications or notifs.Related to notifications or notifs.
Milestone
Description
Adding this issue based on @jleandroperez's comments in this PR here.
In WPiOS we do a bit of a special treatment here. The scenario that prompted this was (afaik somehow) like this:
- Slow connection
- Notifications list gets onscreen
- User pushes the Details for "Notification A"
- Back to the Notes List
- [Time Elapsed]
- "Mark as Read" result comes back
At that point you'd see, out of the blue, the notification getting marked as read. What we do, to attempt to improve the UX, is to optimistically update the Stored Notification.
That way, when you hit (4), the note is "marked as read" locally.
If the remote call fails, we simply invalidate the cache (just set an invalid Note.hash value). This way, next time we sync, the "Optimistic OP" would be reverted.
Metadata
Metadata
Assignees
Labels
feature: notificationsRelated to notifications or notifs.Related to notifications or notifs.