Notifications - Unseen notifications marked as seen about wp-calypso HOT 4 OPEN

Testing more on the old calypso masterbar, p2, and network admin:

• Tab open, Notification made, dot comes on.
• Requests to notifications show a last-seen-time that is consistent and doesn't change as this gets rerequested.

• Reload the tab.
• Initial requests to notifications have the same last-seen-time as the previous load.
• A request goes out to mark notifications as seen and update the last-seen-time.

• notif requests after that have the updated last seen time and the dot goes away.

• When reloading with no new notifications, the /seen never goes out. It seems that these components are always sending a request to /seen when they load IF there is an unseen notification.

Testing on global sidebar at global level: the /seen never gets sent out and this continues to work as expected.

from wp-calypso.

@fullofcaffeine I see you are the most recent contributor to the apps/notifications readme, if you might have any ideas about the above or who to ping. We have the post request for /notifications/seen going out when it shouldn't be.

I tried building calypso dev, sync'd the notes build to my sandbox, sandboxed widgets, etc. and have been trying to test around this, but this is weird. I only see 1 place that calls to sendLastSeenTime which seems is responsible for that post request, but I don't see it called when debugging locally. I can see the updateLastSeenTime debugging commands go off in general use, but when the problematic request goes out none of these debug lines fire. Further I can update that function to change the URL to something other than /notifications/seen and build/sync changes, and the call to /notifications/seen still goes off in the network tab. 🤷‍♀️ 😵

I feel like im missing some steps in getting the development environment for this setup correctly.

from wp-calypso.

I tried building calypso dev, sync'd the notes build to my sandbox, sandboxed widgets, etc. and have been trying to test around this

It seems like there is something related to sandboxing the selected site? This is a really weird end of day find, as it doesn't seem to make any sense but also is consistent in testing.

• Testing on /home
• build and sync the apps/notifications to your sandbox. You don't need any changes, just a sync'd build.
• sandbox public-api, widgets, AND the site you have selected in my-home
• I can no longer reproduce the issue.

As soon as i unsandbox the selected site URL, the issue comes back. Similarly, if I remove the sync'd app build from my sandbox the issue comes back as well, even though that sync'd build had no actual code changes.

The part about the selected site is WEIRD - but it does fit with my initial findings above regarding the discrepency between test B and E - its the same component and sidebar... but at the no site selected level there is no bug, and at the site selected level there is...

from wp-calypso.

A clarifying Update!

Im finding that the problem within calypso is due to notifications from within the site preview. This explains why my local debugging in calypso wasnt working, and explains the discrepency between B and E in the initial tests. If i remove the site preview component from my home, i can no longer repro the issue.

So the problem seems to be within the standalone section of the notes app. I was focusing deeper inside the Panel section since that is all that calypso uses and the bug was reproducible there. Important note - the problem isn't specifically that notes are loaded in site previews, this problem happens everywhere the standalone notes app is loaded. I will continue to investigate more.

from wp-calypso.