Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Browser Notifications: Wrong date on the PN received in Android #88237

Open antonis opened 8 months ago

antonis commented 8 months ago

Quick summary

The date appearing on the browser push notification on Android devices shows the wrong year (1969-1970)

Screenshot 2024-03-06 at 3 54 33 PM

Steps to reproduce

  1. Login from an Android device
  2. Install the web app
  3. Enable browser notifications
  4. Trigger a notification
  5. Notice the date of the notification
Installing the web app Enable notifications Web vs Native notifications
Screenshot_20240306_154155 Screenshot_20240306_154356 Screenshot_20240306_154532

What you expected to happen

The correct date is displayed on the notification

What actually happened

A date with the wrong year is displayed

Impact

All

Available workarounds?

No but the platform is still usable

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

antonis commented 8 months ago

This is related with the mobile report https://github.com/wordpress-mobile/WordPress-Android/issues/16757

lsl commented 3 months ago

Not clear to me who owns this, looks like mobile may have?

The FG page PCYsg-8gV-p2 is a place to start.

The code is in a mu-plugin fbhepr%2Skers%2Sjcpbz%2Sjc%2Qpbagrag%2Szh%2Qcyhtvaf%2Schfu%2Qabgvsvpngvbaf.cuc%3Sn%3Qgehr%26e%3Q925o1po6%26u%3Q158%23759-og and corresponding lib fbhepr%2Skers%2Sjcpbz%2Sjc%2Qpbagrag%2Syvo%2Schfu%2Qabgvsvpngvbaf%2S-og

Next steps: properly repro the issue, the 1969/70 date is telling we probably just don't send the datetime of the notification. It's possible the mobile app should not be trying to display this.

lsl commented 3 months ago

Let's try and fix this on the push side if we can but I'm not seeing how this should be a high priority issue.