From 02c2a0808f455c67e7ed4dfe0797b166fa5ccc1a Mon Sep 17 00:00:00 2001
From: John Curcio <john@curcio.dev>
Date: Thu, 5 Dec 2019 13:19:10 -0300
Subject: [PATCH] Add publication date to medium liquid tag (#4961) (#5002)
 [deploy]

* Add publication date to medium liquid tag

* Add time tag to publication date and fix timezone issue
---
 app/services/medium_article_retrieval_service.rb      | 11 +++++++++++
 app/views/liquids/_medium.html.erb                    |  4 ++--
 spec/liquid_tags/medium_tag_spec.rb                   |  2 ++
 .../services/medium_article_retrieval_service_spec.rb |  2 ++
 4 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/app/services/medium_article_retrieval_service.rb b/app/services/medium_article_retrieval_service.rb
index fada28ed0..071baba13 100644
--- a/app/services/medium_article_retrieval_service.rb
+++ b/app/services/medium_article_retrieval_service.rb
@@ -17,13 +17,24 @@ class MediumArticleRetrievalService
     reading_time = page.at("meta[name='twitter:data1']")["value"]
     author = page.at("meta[name='author']")["content"]
     author_image = page.at("img[alt='#{author}']")["src"]
+    published_time = page.at("meta[property='article:published_time']")["content"]
 
     {
       title: title,
       author: author,
       author_image: author_image,
       reading_time: reading_time,
+      published_time: published_time,
+      publication_date: publication_date(published_time),
       url: url
     }
   end
+
+  private
+
+  def publication_date(published_time)
+    Time.zone.parse(published_time).strftime("%b %-d, %Y")
+  rescue ArgumentError, NoMethodError => e
+    Rails.logger.error("#{published_time} is not a valid date: #{e}")
+  end
 end
diff --git a/app/views/liquids/_medium.html.erb b/app/views/liquids/_medium.html.erb
index 3bce4d991..d433a6417 100644
--- a/app/views/liquids/_medium.html.erb
+++ b/app/views/liquids/_medium.html.erb
@@ -1,13 +1,13 @@
 <div class='ltag__link'>
   <a href='<%= response[:url] %>' class='ltag__link__link'>
     <div class='ltag__link__pic'>
-      <img src='<%= response[:author_image] %>' alt='<%= response[:author] %>'/>
+      <img src='<%= response[:author_image] %>' alt='<%= response[:author] %>' />
     </div>
   </a>
   <a href='<%= response[:url] %>' class='ltag__link__link'>
     <div class='ltag__link__content'>
       <h2><%= response[:title] %></h2>
-      <h3><%= response[:author] %> ・ <%= response[:reading_time] %></h3>
+      <h3><%= response[:author] %> ・ <time datetime='<%= response[:published_time] %>'><%= response[:publication_date] %></time> ・ <%= response[:reading_time] %></h4>
       <div class="ltag__link__servicename">
         <img src="<%= asset_path("medium_icon.svg") %>" alt="Medium Logo" aria-label="Medium Logo" />
         <%= ApplicationController.helpers.get_host_without_www response[:url] %>
diff --git a/spec/liquid_tags/medium_tag_spec.rb b/spec/liquid_tags/medium_tag_spec.rb
index c41a9e2f3..241a28a59 100644
--- a/spec/liquid_tags/medium_tag_spec.rb
+++ b/spec/liquid_tags/medium_tag_spec.rb
@@ -13,6 +13,8 @@ RSpec.describe MediumTag, type: :liquid_tag do
       author: "Edison",
       author_image: "https://cdn-images-1.medium.com/fit/c/120/120/1*qFzi921ix0_kkrFMKYgELw.jpeg",
       reading_time: "4 min read",
+      published_time: "2018-11-03T09:44:32.733Z",
+      publication_date: "Nov 3, 2018",
       url: "https://medium.com/@edisonywh"
     }
   end
diff --git a/spec/services/medium_article_retrieval_service_spec.rb b/spec/services/medium_article_retrieval_service_spec.rb
index 2e803983e..e7b29a1b9 100644
--- a/spec/services/medium_article_retrieval_service_spec.rb
+++ b/spec/services/medium_article_retrieval_service_spec.rb
@@ -7,6 +7,8 @@ RSpec.describe MediumArticleRetrievalService, type: :service, vcr: {} do
       author: "Edison Yap",
       author_image: "https://miro.medium.com/fit/c/96/96/1*qFzi921ix0_kkrFMKYgELw.jpeg",
       reading_time: "4 min read",
+      published_time: "2018-11-03T09:44:32.733Z",
+      publication_date: "Nov 3, 2018",
       url: "https://medium.com/@edisonywh/my-ruby-journey-hooking-things-up-91d757e1c59c"
     }
   end