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.
Content-Length: 235588 | pFad | http://phabricator.wikimedia.org/T231102
s• Mholloway | |
Aug 23 2019, 5:31 PM |
F31478025: CAT-mails.jpg | |
Dec 16 2019, 5:13 PM |
F31018045: Screen Shot 2019-11-05 at 1.51.33 PM.png | |
Nov 5 2019, 11:22 PM |
F31011968: Screen Shot 2019-11-05 at 10.02.37 AM.png | |
Nov 5 2019, 6:27 PM |
F31011745: Screen Shot 2019-11-05 at 10.03.36 AM.png | |
Nov 5 2019, 6:27 PM |
F31011754: Screen Shot 2019-11-05 at 10.02.29 AM.png | |
Nov 5 2019, 6:27 PM |
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.
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | egardner | T231102 Notify the uploader when a newly uploaded image has machine-aided depicts suggestions ready for review | |||
Resolved | None | T233031 Design elements for CAT tagging ready-for-review notifications |
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?
Sorry for the confusion, my asset question was related to the UploadWizard task T234960, getting my tickets confused over here.
I'm going to see how far I can get following these guidelines: https://m.mediawiki.org/wiki/Extension:Echo/Creating_a_new_notification_type
Change 545117 had a related patch set uploaded (by Eric Gardner; owner: Eric Gardner):
[mediawiki/extensions/MachineVision@master] WIP: Add echo notification support
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
Change 546917 had a related patch set uploaded (by Eric Gardner; owner: Eric Gardner):
[mediawiki/extensions/MachineVision@master] Add Echo notifications
Ok, made some progress on this. Here are some screenshots of the current appearance/behavior of notifications:
Popup view, non-bundled
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:
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
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:
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
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
Change 550512 merged by Mholloway:
[mediawiki/extensions/MachineVision@wmf/1.35.0-wmf.5] Add Echo notifications
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
Fetched URL: http://phabricator.wikimedia.org/T231102
Alternative Proxies: