Skip to content
This repository has been archived by the owner on Feb 29, 2020. It is now read-only.

Implement recommendation highlight #870

Closed
edwindotcom opened this issue Jun 21, 2016 · 7 comments
Closed

Implement recommendation highlight #870

edwindotcom opened this issue Jun 21, 2016 · 7 comments
Assignees

Comments

@edwindotcom
Copy link

edwindotcom commented Jun 21, 2016

This ticket should replace one of the tiles in the Highlights section with a "recommended" tile, if possible.

We should be able to use the new pocket recommendation engine to do this.

[Edit by @k88hudson for clarity]

spec:
https://trello.com/c/TQcZZhMe

@edwindotcom edwindotcom changed the title Implement recommendation of evergreen content Implement recommendation highlight Jun 22, 2016
@edwindotcom edwindotcom added this to the Columbia (Jun 25) milestone Jun 22, 2016
@k88hudson
Copy link
Contributor

k88hudson commented Jun 24, 2016

We still need to work out:

  • What should be the source of the recommendation? Should it be related to an existing tile that is also shown? Should it replace the tile it is related to? Should it be a general "trending" recommendation?
  • What if a recommendation is not available from the designated source? Should we replace it with a general recommendation? Should we just not show it?
  • How frequently should recommendations be shown? All the time when we have one? Only sometimes?

Also note: we should try running this as an experiment.

If you want, I can try something like replacing one of the tiles with its recommendation, and we can look at it/go from there

cc @bryanbell @aaronrbenson

@k88hudson
Copy link
Contributor

If you want, I can try something like replacing one of the tiles with its recommendation, and we can see how that goes.

@k88hudson
Copy link
Contributor

k88hudson commented Jun 24, 2016

Note: this is also blocked on implementing pocket endpoints in the proxy

mozilla/embedly-proxy#122

@aaronrbenson
Copy link

aaronrbenson commented Jun 29, 2016

new tab - recommended content spec

@k88hudson
Copy link
Contributor

@tspurway I wasnt able to make the meeting on Thursday, did you end up asking about the short URLs?

@k88hudson
Copy link
Contributor

Also thanks for the clarification @aaronrbenson! Looks great

sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 14, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 18, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 18, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 18, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 18, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 19, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 20, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 20, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 20, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit to sarracini/activity-stream that referenced this issue Jul 21, 2016
sarracini added a commit that referenced this issue Jul 22, 2016
feat(content): #870 Turn recommendations on as experiment
@sarracini
Copy link
Contributor

Fixed with #906

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants