Merge pull request #1770 from basecamp/og-update

Opengraph update

Author: jzimdars

app/helpers/cards_helper.rb

@@ -39,9 +39,9 @@ def card_drafted_or_added(card)
 
   def card_social_tags(card)
     tag.meta(property: "og:title", content: "#{card.title} | #{card.board.name}") +
-    tag.meta(property: "og:description", content: format_excerpt(@card&.description, length: 200)) +
-    tag.meta(property: "og:image", content: @card.image.attached? ? "#{request.base_url}#{url_for(@card.image)}" : "#{request.base_url}/app-icon.png") +
-    tag.meta(property: "og:url", content: card_url(@card))
+    tag.meta(property: "og:description", content: format_excerpt(card&.description, length: 200)) +
+    tag.meta(property: "og:image", content: card.image.attached? ? "#{request.base_url}#{url_for(card.image)}" : "#{request.base_url}/opengraph.png") +
+    tag.meta(property: "og:url", content: card_url(card))
   end
 
   def button_to_remove_card_image(card)

app/views/public/boards/show.html.erb

@@ -3,12 +3,12 @@
 <% content_for :head do %>
   <%= tag.meta property: "og:title", content: "#{@board.name} | #{Current.account.name}" %>
   <%= tag.meta property: "og:description", content: format_excerpt(@board&.public_description, length: 200) %>
-  <%= tag.meta property: "og:image", content: "#{request.base_url}/app-icon.png" %>
+  <%= tag.meta property: "og:image", content: "#{request.base_url}/opengraph.png" %>
   <%= tag.meta property: "og:url", content: published_board_url(@board) %>
 
   <%= tag.meta property: "twitter:title", content: "#{@board.name} | #{Current.account.name}" %>
   <%= tag.meta property: "twitter:description", content: format_excerpt(@board&.public_description, length: 200) %>
-  <%= tag.meta property: "twitter:image", content: "#{request.base_url}/app-icon.png" %>
+  <%= tag.meta property: "twitter:image", content: "#{request.base_url}/opengraph.png" %>
   <%= tag.meta property: "twitter:card", content: "summary_large_image" %>
 <% end %>