Merge branch 'articles-metadata'

src/app/routes/v1/articles.js

@@ -3,40 +3,69 @@ const Router = require('koa-router');
 
 const Article = require('../../../model/article');
 const getArticleSources = require('../../../articles/get-article-sources');
+const middleware = require('../../middleware');
 const transformArticle = require('./util/transform-article');
 
+const parseBoolean = booleanString => {
+  if (
+    booleanString === undefined ||
+    booleanString === null ||
+    booleanString.trim().length === 0
+  ) {
+    return undefined;
+  }
+
+  return booleanString === 'true';
+};
+
 const createRouter = () => {
   const router = new Router({ prefix: '/articles' });
 
-  router.get('/', async ({ response, request }, next) => {
-    const sources = await getArticleSources();
-    const page = request.query.page || 1;
+  router.get(
+    '/',
+    middleware.pagination({
+      defaultLimit: 12,
+      maxLimit: 50,
+      maxPage: Infinity,
+    }),
+    async ({ pagination, response, request }, next) => {
+      const sources = await getArticleSources();
+      const editorsChoice = parseBoolean(request.query.editorsChoice);
+
+      const feed = _.findKey(
+        sources,
+        source => source.slug === request.query.source,
+      );
 
-    const feed = _.findKey(
-      sources,
-      source => source.slug === request.query.source,
-    );
+      const query = _.pickBy(
+        {
+          feed: request.query.source ? feed : undefined,
+          editorsChoice:
+            editorsChoice === true || editorsChoice === undefined
+              ? editorsChoice
+              : { $in: [false, null] },
+        },
+        i => i !== undefined,
+      );
 
-    const articles = await Article.paginate(
-      request.query.source ? { feed } : {},
-      {
+      const articles = await Article.paginate(query, {
         sort: { date: -1 },
         lean: true,
-        limit: 12,
-        page,
-      },
-    );
+        limit: pagination.limit,
+        page: pagination.page,
+      });
 
-    response.body = {
-      articles: _(articles.docs).map(_.partial(transformArticle, sources)),
-      limit: articles.limit,
-      page: parseInt(articles.page, 10),
-      pageCount: articles.pages,
-      total: articles.total,
-    };
+      response.body = {
+        articles: _(articles.docs).map(_.partial(transformArticle, sources)),
+        limit: articles.limit,
+        page: parseInt(articles.page, 10),
+        pageCount: articles.pages,
+        total: articles.total,
+      };
 
-    await next();
-  });
+      await next();
+    },
+  );
 
   router.get('/:feedSlug/:articleSlug', async ({ response, params }, next) => {
     const { feedSlug, articleSlug } = params;

src/app/routes/v1/util/transform-article.js

@@ -5,6 +5,7 @@ const transformArticle = (sources, article) => {
 
   return {
     ..._.pick(article, ['date', 'id', 'slug', 'summary', 'title', 'url']),
+    imageUrl: _.get(article, 'metadata.og:image', null),
     source,
   };
 };

src/model/article.js

@@ -7,6 +7,7 @@ const schema = mongoose.Schema({
   date: Date,
   feed: String,
   guid: String,
+  metadata: mongoose.Schema.Types.Mixed,
   slug: String,
   summary: String,
   title: String,