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

Notifications: Opening WP notif from Android system notification doesn't appear as Read #4889

Closed
hoverduck opened this issue Dec 8, 2016 · 1 comment

Comments

@hoverduck
Copy link
Contributor

Steps to reproduce the behavior

  1. Spawn a WP.com notification, such as by leaving a comment on your site as another user
  2. Open the Android notification, which should launch the app to the WP notification
  3. Back out to the notifications list

Expected behavior

To have the notification marked as read

Actual behavior

The notification still appears unread. It's a display issue only, looking at the notification on another device it appears read (or if you refresh the in-app notifications list)

Tested on Nexus 6P, Android 7.1.1, WP alpha-30
@mzorz
Copy link
Contributor

mzorz commented Dec 16, 2016

Been investigating this, for note keeping:

We have a situation of race condition where when you tap on a notification and read a note (by opening the note detail activity) we mark it read locally, but the act of tapping on it brings the app to the foreground triggering the notificaitons update service which brings the whole notifications batch for this user before we can mark it read on the server, thus washing the db and setting the note again as unread. Working on a solution to it.

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

2 participants