Page MenuHomePhabricator

Unable to view notifications in Echo
Closed, ResolvedPublic


Caught by browser test and now blocking all MobileFrontend merges:

Notifications overlay is broken on mobile site.
The error reported in the JS console is:
TypeError: Cannot read property 'bind' of undefined
and it seems to have been introduced by 1575e2bb7a2f686ba4fba71c19bfd70b4c18abc7

Side note: Should MobileFrontend run our browser test suite - this has happened a few times now.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 312105 had a related patch set uploaded (by Jdlrobson):
Revert "Convert 'seenTime' to a global property"

Jdlrobson triaged this task as Unbreak Now! priority.Sep 21 2016, 8:37 PM

Dealing with this right now. There's a small change we forgot to make when fixing global seen time - I'm submitting now. There's no need to revert the whole seenTime change.

Change 312112 had a related patch set uploaded (by Mooeypoo):
Followup I67bcc4b34: Update call to global seenTime

Change 312112 merged by jenkins-bot:
Followup I67bcc4b34: Update call to global seenTime

Fixed for me locally and seems to be passing tests. Can you confirm this is fixed, @Jdlrobson ? Can we abandon the revert patch?

Checked in betalabs mobile

Change 312105 abandoned by Jdlrobson:
Revert "Convert 'seenTime' to a global property"

Feel free to abandon these in future. I never get offended by abandons and sometimes forget. :) thanks again for fixing so quickly!

Last build passed: (we've been having some turbulence) but this looks fixed to me.