Skip to content

Conversation

mickeyn
Copy link
Contributor

@mickeyn mickeyn commented Jun 2, 2017

Make use of the new /favorite/recent API endpoint, replacing
the Elasticsearch query in WEB.

This change also moves the author matching code away from the controller, into the document module.

Note: this PR in API needs to be merged first: metacpan/metacpan-api#675

Make use of the new /favorite/recent API endpoint, replacing
the Elasticsearch query in WEB.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 44.247% when pulling 0b3bb03 on mickey/favorite_recent_endpoint into 1f94ec4 on master.

@mickeyn mickeyn changed the title Replace favorite recent query with endpoint call [DO NOT MERGE YET] Replace favorite recent query with endpoint call Jun 2, 2017
@mickeyn mickeyn changed the title [DO NOT MERGE YET] Replace favorite recent query with endpoint call [DO NOT MERGE YET - see note] Replace favorite recent query with endpoint call Jun 2, 2017
@mickeyn mickeyn changed the title [DO NOT MERGE YET - see note] Replace favorite recent query with endpoint call Replace favorite recent query with endpoint call Jun 2, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 44.247% when pulling 8d9d16a on mickey/favorite_recent_endpoint into 5796955 on master.

Copy link
Member

@oalders oalders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@oalders oalders merged commit fc0ae58 into master Jun 2, 2017
@oalders oalders deleted the mickey/favorite_recent_endpoint branch June 2, 2017 19:03
@coveralls
Copy link

Coverage Status

Coverage remained the same at 44.247% when pulling 0948883 on mickey/favorite_recent_endpoint into ed913b7 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants