redirect skipped article instead of 404ing it (#424)

redirect skipped article instead of 404ing it

server/controllers/amp-page.js

@@ -3,7 +3,16 @@
 const assembleArticle = require('../lib/article/assemble');
 const analytics = require('../lib/analytics');
 
+// HACK: redirect certain articles to ft.com
+const articlesToSkip = [
+	'263615ca-d873-11e9-8f9b-77216ebe1f17', // general election poll tracker: contains image that shouldn't be cached
+];
+
 module.exports = (req, res, next) => {
+	if(articlesToSkip.includes(req.params.uuid)) {
+		return res.redirect(`https://www.ft.com/content/${req.params.uuid}`);
+	}
+
 	assembleArticle(req.params.uuid, {
 		development: req.app.isDevelopment,
 		production: req.app.isProduction,

server/lib/article/assemble.js

@@ -60,11 +60,6 @@ const extraArticleData = (article, options) => promiseAllObj({
 	css: getCSS(article, options),
 }).then(extra => Object.assign(article, extra));
 
-// HACK: hide certain articles from the AMP cache
-const articlesToSkip = [
-	'263615ca-d873-11e9-8f9b-77216ebe1f17', // general election poll tracker: contains image that shouldn't be cached
-];
-
 const assembleArticle = (uuid, options) => {
 	options = Object.assign({}, environmentOptions, options);
 
@@ -74,7 +69,6 @@ const assembleArticle = (uuid, options) => {
 				if(response
 					&& (!response.originatingParty || response.originatingParty === 'FT')
 					&& (!response.type || response.type === 'article')
-					&& (!articlesToSkip.includes(uuid))
 				) {
 					return response;
 				}