Page MenuHomePhabricator

Link http://recommend.wmflabs.org from CX Suggestions as fallback
Closed, ResolvedPublic1 Estimated Story Points

Description

For recommending articles, CX has suggestion feature now in basic form. We do not have suggestion for all language pairs. But if we do not have suggestions to show, it might be good idea to give a link to http://recommend.wmflabs.org

http://recommend.wmflabs.org also does not provide suggestion for all language pairs, So we might need to do it conditionally.

The idea is to link it here:

screenshot2.png (379×1 px, 24 KB)

Event Timeline

santhosh raised the priority of this task from to Needs Triage.
santhosh updated the task description. (Show Details)
santhosh added subscribers: santhosh, leila.
Amire80 subscribed.

Similarly to the "Magnus link", in the dashboard sidebar, right?

Amire80 triaged this task as Medium priority.Sep 16 2015, 9:27 AM
Amire80 set Security to None.
Amire80 moved this task from Needs Triage to CX6 on the ContentTranslation board.

Similarly to the "Magnus link", in the dashboard sidebar, right?

No. Where we show "No suggestions". See screenshot above.

We can add a link as a next step forward when showing the empty state:

dashboard-empty-suggestions-external.png (718×1 px, 74 KB)

@santhosh do you have a sense about which sprint this task will be addressed in? We'd like to get a sense of where we are so we can plan better on our end.

Change 248852 had a related patch set uploaded (by Amire80):
Add a link to the experimental recommend tool

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

Change 248853 had a related patch set uploaded (by Amire80):
Change the alignment of the empty list messages

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

The change to show the link is committed, but I need the image for the bulb according to the design.

The graphic elements are these:

lightbulb-b.png (33×33 px, 982 B)

We can add a link as a next step forward when showing the empty state:

dashboard-empty-suggestions-external.png (718×1 px, 74 KB)

@Pginer-WMF, can you confirm you meant to left align the text in this? This is a style change from centered text we have already.( This is for reviewing https://gerrit.wikimedia.org/r/#/c/248853)

@Pginer-WMF, can you confirm you meant to left align the text in this? This is a style change from centered text we have already.( This is for reviewing https://gerrit.wikimedia.org/r/#/c/248853)

With the mockup I was focusing mainly on the copy, the alignment change was unintended.
I gave it a try and keeping the center alignment seems to work better in this context:

dashboard-empty-suggestions-external.png (1×1 px, 80 KB)

Change 248853 abandoned by Amire80:
Change the alignment of the empty list messages

Reason:
Pau decided that this is not needed.

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

Change 248852 merged by jenkins-bot:
Add a link to the experimental recommend tool

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

Arrbee subscribed.

Next up is T111028 and this url will be removed when we have the recommendations included.

Arrbee raised the priority of this task from Medium to Unbreak Now!.Nov 4 2015, 6:38 AM
Arrbee lowered the priority of this task from Unbreak Now! to Medium.
Arrbee moved this task from In Review to Done on the LE-CX7-Sprint 2 board.