Page MenuHomePhabricator

Notify the uploader when a newly uploaded image has machine-aided depicts suggestions ready for review
Closed, ResolvedPublic

Description

Upon successfully adding depicts suggestions to the DB for a newly uploaded image, we should emit an event to notify the uploader that the image has depicts suggestions needing review.

See subtask for design elements.

Event Timeline

Mholloway renamed this task from Notify the user when an uploaded image has machine-aided depicts suggestions ready for review to Notify the uploader when a newly uploaded image has machine-aided depicts suggestions ready for review.Aug 23 2019, 5:32 PM
Mholloway moved this task from Ready for dev to Backlog on the MachineVision board.
LGoto triaged this task as Medium priority.Aug 28 2019, 3:44 PM
LGoto moved this task from Needs triage to Upcoming on the Product-Infrastructure-Team-Backlog board.
Mholloway raised the priority of this task from Medium to High.Oct 15 2019, 10:08 PM
Mholloway moved this task from Needs design to Ready for dev on the MachineVision board.
Mholloway updated the task description. (Show Details)

I can get a patch ready for this pretty quickly, but do we have assets (vector graphics) and copy for this screen now? @PDrouin-WMF any updates here?

@egardner - yep, see this subtask (T233031) for the svg (link to commons file) and the copy (link to google doc). lmk if you have any questions about either!

Sorry for the confusion, my asset question was related to the UploadWizard task T234960, getting my tickets confused over here.

Change 545117 had a related patch set uploaded (by Eric Gardner; owner: Eric Gardner):
[mediawiki/extensions/MachineVision@master] WIP: Add echo notification support

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

pinging @Cparle and @Mholloway for code review when this is ready (later this week probably).

Change 545117 abandoned by Eric Gardner:
WIP: Add echo notification support

Reason:
will replace with new attempt

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

Change 546917 had a related patch set uploaded (by Eric Gardner; owner: Eric Gardner):
[mediawiki/extensions/MachineVision@master] Add Echo notifications

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

Ok, made some progress on this. Here are some screenshots of the current appearance/behavior of notifications:

Popup view, non-bundled

Screen Shot 2019-11-05 at 10.03.36 AM.png (644×1 px, 103 KB)

This is what the user will see if they only have a single unread notification about Suggested Tags. In this case, the notification heading is file-specific, and the body includes the longer message about the tool (which is truncated until the user views it in their notifications dashboard). Clicking the notification goes to the user tab of Special:SuggestedTags.

Popup view, bundled:

Screen Shot 2019-11-05 at 10.02.29 AM.png (782×1 px, 106 KB)

This is what the user sees when there are multiple unread "suggested tags ready" notifications. The overall heading is a generic "Suggested tags are reaady for review" message. When expanded, the "compact heading" for each message in the bundle will indicate the filename of the file in question. Clicking any of these notifications will link the user to the user tab of Special:SuggestedTags, as above.

Inbox view

Screen Shot 2019-11-05 at 10.02.37 AM.png (1×2 px, 336 KB)

Messages do not bundle on the user's Notifications dashboard. Notifications are displayed in the same way as the non-bundled popup-view above. Now the full body text is visible in a non-truncated form.

Email view
Still figuring out how to enable / test this, not sure exactly what the email version includes.

Does this seem mostly correct? I've taken a few small liberties with the initial content outline with an eye to providing a little more useful info to the user.

I've updated the "bundled view" to not be expandable per @Ramsey-WMF's suggestion – the concern is that batch uploaders may be confronted with way too much information if we show a line for each individual file.

If a user has multiple unread "suggestions-ready" notifications, the popup view now looks like this:

Screen Shot 2019-11-05 at 1.51.33 PM.png (458×1 px, 70 KB)

Changing this back to the earlier behavior is as simple as flipping a boolean value from false to true, so let me know if that needs to change again.

Change 546917 merged by jenkins-bot:
[mediawiki/extensions/MachineVision@master] Add Echo notifications

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

Change 550512 had a related patch set uploaded (by Mholloway; owner: Eric Gardner):
[mediawiki/extensions/MachineVision@wmf/1.35.0-wmf.5] Add Echo notifications

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

Change 550512 merged by Mholloway:
[mediawiki/extensions/MachineVision@wmf/1.35.0-wmf.5] Add Echo notifications

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

Email notifications don't bundle, which can lead to an inbox full of messags.

CAT-mails.jpg (1×640 px, 86 KB)

I've added a new ticket to track this issue at T240878 – feel free to add this screenshot there.

Since the on-site notification system is working as expected, will close this ticket and focus on T240878