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 aren't showing up without a page refresh #97767

Open
allilevine opened this issue Dec 24, 2024 · 8 comments
Open

Notifications aren't showing up without a page refresh #97767

allilevine opened this issue Dec 24, 2024 · 8 comments
Assignees
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Experiment] AI labels added [Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. [Feature Group] User Interaction & Engagement Tools and features for site owners to share, promote, and manage engagement with their audiences. [Feature] Notifications Site notifications. [Pri] Normal Schedule for the next available opportuinity. [Pri] TBD Review and assign an appropriate [Pri] label as soon as possible. [Status] Auto-allocated [Type] Bug

Comments

@allilevine
Copy link
Member

allilevine commented Dec 24, 2024

User report: 9203237-zd-a8c
Slack thread: p1734864094501249-slack-C029GN3KD

Screen.Recording.2024-12-22.at.8.44.48.PM.mov

Notifications aren't showing up without a page refresh. This is happening for some users but not others, and sometimes with posts, sometimes with comments. Notifications are implemented both in Calypso and for wp-admin (iframed). I see network requests being fired from both when I open the notifications panel.

In Calypso see: https://github.com/Automattic/wp-calypso/blob/trunk/apps/notifications/src/panel/rest-client/index.js#L485

@github-actions github-actions bot added [Pri] TBD Review and assign an appropriate [Pri] label as soon as possible. [Feature Group] User Interaction & Engagement Tools and features for site owners to share, promote, and manage engagement with their audiences. [Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. [Feature] Notifications Site notifications. labels Dec 24, 2024
Copy link

OpenAI suggested the following labels for this issue:

  • [Feature Group] User Interaction & Engagement: The issue involves notifications, which are a key part of how users interact with and engage with the content on the platform.
  • [Feature] Notifications: This label is directly relevant as the issue specifically addresses a problem with notifications not appearing.
  • [Feature] Calypso & wp-admin Navigation: Notifications are implemented in both Calypso and wp-admin, making this feature relevant to the issue at hand.

Copy link

This issue could use some more labels, to help prioritize and categorize our work. Could you please add at least a [Type], a [Feature], and a [Pri] label?

Copy link

Support References

This comment is automatically generated. Please do not edit it.

  • 9203237-zen

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Dec 24, 2024
@allilevine allilevine added the [Pri] Normal Schedule for the next available opportuinity. label Dec 24, 2024
@ariel-maidana
Copy link

The site in 9203237-zd-a8c is a simple site on the Premium plan.

@ariel-maidana
Copy link

Using a test site, I was able to replicate this in:

  • Sites screen.
  • Calypso home.
  • WP Admin home. This one was inconsistent: in one of the tests, the notification appeared without reloading the page, while in the second test, I had to reload the page to see it.

@allilevine allilevine self-assigned this Dec 25, 2024
@allilevine allilevine moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Dec 25, 2024
@allilevine
Copy link
Member Author

I'm testing in Calypso with localStorage.debug = 'notifications:panel' enabled in the console, and I see comment notifications showing up without a refresh, but post notifications needing a refresh. The debug is showing the correct values. I'll keep investigating.

@allilevine
Copy link
Member Author

allilevine commented Dec 26, 2024

Today comment notifications aren't always showing up. With more debugging I see that we're supposed to get new notification messages in real-time via a WebSocket connection to wss://public-api.wordpress.com/pinghub/wpcom/me/newest-note-data, but the messages aren't coming through. We fall back to polling for new notifications, so that explains why it sometimes works.

Next step is to debug on the server side to find out why the messages aren't coming through. fbhepr%2Skers%2Sjcpbz%2Sjc%2Qpbagrag%2Szh%2Qcyhtvaf%2Sabgrf%2Sabgrf%2Qubbxf.cuc%3Se%3Q7s7s45ro%2349-og

@allilevine
Copy link
Member Author

I'm hitting a wall with debugging this, asked here: p1735662659279979-slack-C03N25JPCE4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Experiment] AI labels added [Feature] Calypso & wp-admin Navigation All navigation in Calypso and wp-admin, and the unified transitions between the two. [Feature Group] User Interaction & Engagement Tools and features for site owners to share, promote, and manage engagement with their audiences. [Feature] Notifications Site notifications. [Pri] Normal Schedule for the next available opportuinity. [Pri] TBD Review and assign an appropriate [Pri] label as soon as possible. [Status] Auto-allocated [Type] Bug
Projects
Development

No branches or pull requests

3 participants