Created attachment 16344
Mark all as read button gone missing

Attached:

• Bug 70297 has been marked as a duplicate of this bug. ***

• This bug has been marked as a duplicate of bug 70329 ***

Jon, Bug 70329 - "Message notifications not working" - doesn't sound like this bug, so I'm un-dupe-ing.

I am seeing "Mark all as read" on ee-flow and beta labs when I have notifications from Flow in the Messages pane. Looking at the attached screenshot, Messages(0) means you have zero unread message notifications and they all look gray, so there is indeed no "Mark all as read".

(The old way "Mark as read" worked is if you had more alerts than the non-scrolling flyout could show, "Mark all as read" would mark all the notifications as read, not just the ones appearing in the flyout.)

Thanks S, so this is a feature not a bug.

The failure is due to the other bug.

So it *is* a bug. :-)

I'll leave the failing test as is for now then.

Bump because the test is still failing: https://integration.wikimedia.org/ci/view/BrowserTests/view/-All/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/52/testReport/(root)/Unread%20messages/Mark_all_as_unread/

This is failing for another reason - the default tab was changed and the test was not updated - there needs to be a new step "And I switch to the messages tab"

Change 159685 had a related patch set uploaded by Spage:
QA: update Echo messages test

https://gerrit.wikimedia.org/r/159685

Change 159685 abandoned by Cmcmahon:
QA: update Echo messages test

Reason:
overtaken by history: https://gerrit.wikimedia.org/r/#/c/160477/

https://gerrit.wikimedia.org/r/159685

Reopening for regression bug, the "Mark all as read" button is missing again in beta labs, although it functions normally in test2wiki right now.

https://integration.wikimedia.org/ci/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/92/testReport/junit/(root)/Scenarios%20that%20trigger%20notifications/Mark_all_as_unread/

At a quick glance, it seems the
"And I have a Flow message that triggers an alert notification"

isn't happening. The Echo notification badge count doesn't go to 2, there's no recent message in the messages tab.

For that matter, there's no recent alert in the alerts tab. Alerts are marked on view, but I would expect to see something more recent in there than "1 day ago"

It might be delayed notifications: to check see if 'Selenium user firefox' has more recent messages.

I'm not getting any Flow Notifications at beta.wmflabs. I replied to myself and thanked myself (via a testaccount) at http://en.wikipedia.beta.wmflabs.org/wiki/Topic:S3bmb9jyd5l7c34i - I got the Thank notification instantly, but the Flow-reply notifications have not shown up after 1 hour.

I filed bug 71484 for the general problem of Flow notifications on beta labs.

This is still an issue. If you don't care about the browser tests you should probably disable them... (but I highly encourage you to rethink this if it is the case)

I've flagged this to be estimated; we should definitely highly consider it for our next sprint.

Where are you missing the "mark all as read" link? I can see it, on both mediawiki and enwiki.

Note that it doesn't appear on the Alerts tab, and does appear on the Messages tab.

Here's screenshots I just took from enwiki.

The recent build https://integration.wikimedia.org/ci/view/BrowserTests/view/Echo+Flow/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/528/ is green.

The scenario Scenario: Mark all as unread # features/messages.feature:4 is present and seems to be passing.

What was checked:

• re-check the 'Mark all as read' button in beta/production in Chrome/FF/Safari
• successfully re-run the related tests(Echo/tests/browser/features/messages.feature) both locally and in beta env
• checked the last 10 builds(https://integration.wikimedia.org/ci/view/BrowserTests/view/Echo+Flow/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/). The test scenario with 'Mark all as read' has failed only once but for a different reason.