Wp-calypso: Notifications: filters redirect to old URL in Windows

[davewhitley]

Steps to reproduce

1. Starting at URL: https://wordpress.com/
2. Click Notifications bell icon
3. Click on a filter
4. It incorrectly goes to https://wordpress.com/notifications
   

   Browser / OS version

• IE 11 Windows 7
• Edge 20 Windows 10
  

  Screenshot / Video

[dmsnell]

nasty - thanks for finding @drw158

[lancewillett]

Could we start redirecting /notifications now to auto-route to Calypso?

[lancewillett]

Re-ping @dmsnell on this one since you're working on IE already. :)

[lancewillett]

Re-testing this today with Windows 10 and IE Edge, still happening — needs fixing. Clicking any of the notifications tabs (Unread, Comments, Follows, Likes) redirects the page to https://wpcalypso.wordpress.com/notifications — which is a 404.

I confirmed Chrome and desktop app in same Windows 10 install worked correctly — seems to be only Internet Explorer.

[lancewillett]

Hi @dmsnell — just checking on this one, happy to re-test or confirm a fix when you're ready.

[dmsnell]

@lancewillett I haven't worked on it at all. I may have ran into the same problem when working on integrating my iframe-less notifications client - the master bar was wrapping (at a much higher point in the tree) the notifications bell in an <a> when a <div> would have probably been more appropriate. Something with the way the events were propagating was coming back up to it.

I'm out this week and can try and look into it when I get back unless @rodrigoi or someone knowledgable with the master bar wants to try and fix it before then.

Thanks for the checkup!

[rodrigoi]

@lancewillett @dmsnell I'll look into this. Definitely looks like an event propagation issue.

[lancewillett]

@rodrigoi Any update on this one? It's a few weeks old now.

[rodrigoi]

@lancewillett PR is up for review: https://github.com/Automattic/notifications-client/pull/608 😉

[lancewillett]

Still pending a merge from the notifications client side.