From 178b0a32f0e69eaa029c424322a458989aecfa4b Mon Sep 17 00:00:00 2001
From: Ridhwana <Ridhwana.Khan16@gmail.com>
Date: Thu, 12 Nov 2020 17:02:58 +0200
Subject: [PATCH] Remove information route and add a temporary redirect to
 /about for SEO purposes  (#11373)

* chore: remove information route and add redirect

* chore: update messaging

* fix: remove additional path referenced
---
 app/views/layouts/_nav_menu.html.erb |  4 --
 app/views/pages/information.html.erb | 57 ----------------------------
 config/routes.rb                     |  6 ++-
 3 files changed, 5 insertions(+), 62 deletions(-)
 delete mode 100644 app/views/pages/information.html.erb

diff --git a/app/views/layouts/_nav_menu.html.erb b/app/views/layouts/_nav_menu.html.erb
index d8f7be4eb..83428b319 100644
--- a/app/views/layouts/_nav_menu.html.erb
+++ b/app/views/layouts/_nav_menu.html.erb
@@ -15,7 +15,6 @@
       <a href="<%= new_path %>" class="crayons-link crayons-link--block">Write a Post</a>
       <a href="<%= readinglist_path %>" class="crayons-link crayons-link--block">Reading list</a>
       <a href="<%= user_settings_path %>" class="crayons-link crayons-link--block">Settings</a>
-      <a href="<%= information_path %>" class="crayons-link crayons-link--block" id="second-last-nav-link">Key Links</a>
     </div>
     <div class="p-1">
       <a href="<%= signout_confirm_path %>" class="crayons-link crayons-link--block" id="last-nav-link">Sign Out</a>
@@ -43,9 +42,6 @@
       <a href="<%= about_path %>" class="crayons-link crayons-link--block" id="last-nav-link">
         About <%= community_name %>
       </a>
-      <a href="<%= information_path %>" class="crayons-link crayons-link--block" id="last-nav-link">
-        Learn more
-      </a>
     </div>
   <% end %>
 </div>
diff --git a/app/views/pages/information.html.erb b/app/views/pages/information.html.erb
deleted file mode 100644
index a8d528127..000000000
--- a/app/views/pages/information.html.erb
+++ /dev/null
@@ -1,57 +0,0 @@
-<% title "About #{community_qualified_name}" %>
-
-<%= content_for :page_meta do %>
-  <link rel="canonical" href="<%= app_url("/p/information") %>" />
-  <meta name="description" content="All information about <%= community_qualified_name %>">
-  <%= meta_keywords_default %>
-
-  <meta property="og:type" content="article" />
-  <meta property="og:url" content="<%= app_url("/p/information") %>" />
-  <meta property="og:title" content="All information about <%= community_qualified_name %>" />
-  <meta property="og:image" content="<%= SiteConfig.main_social_image %>" />
-  <meta property="og:description" content="<%= community_qualified_name %> is great!" />
-  <meta property="og:site_name" content="<%= community_qualified_name %>" />
-
-  <meta name="twitter:card" content="summary_large_image">
-  <meta name="twitter:site" content="@<%= SiteConfig.social_media_handles["twitter"] %>">
-  <meta name="twitter:title" content="All information about <%= community_qualified_name %>">
-  <meta name="twitter:description" content="<%= community_qualified_name %> is great!">
-  <meta name="twitter:image:src" content="<%= SiteConfig.main_social_image %>">
-<% end %>
-
-<style>
-  .key-links {
-    font-family: "Helvetica Neue", Helvetica, Arial, "Lucida Grande", sans-serif;
-  }
-
-  .social-links {
-    text-align: center;
-    padding: 40px 0px;
-    filter: invert(0);
-    filter: var(--theme-social-icon-invert, invert(0));
-  }
-</style>
-
-<div class="crayons-layout crayons-layout--limited-l">
-  <div class="crayons-card text-styles text-padding">
-    <h1 class="fs-3xl s:fs-4xl l:fs-5xl fw-bold s:fw-heavy lh-tight mb-4 mt-0">More information about <%= community_name %> 👋</h1>
-
-    <ul>
-      <li><a href="/">Home</a></li>
-      <li><a href="/about">About</a></li>
-      <% if SiteConfig.shop_url.present? %>
-        <li><a href="<%= SiteConfig.shop_url %>"><%= community_name %> Shop</a></li>
-      <% end %>
-      <li><a href="/sponsors">Sponsors</a></li>
-      <li><a href="/faq">FAQ</a></li>
-      <li><a href="/privacy">Privacy Policy</a></li>
-      <li><a href="/terms">Terms of Use</a></li>
-      <li><a href="/contact">Contact</a></li>
-      <li><a href="/security">Vulnerabilities/Bug Bounty</a></li>
-      <li><a href="/code-of-conduct">Code of Conduct</a></li>
-      <li><a href="/downloads">Get the mobile app</a></li>
-    </ul>
-
-    <%= render partial: "layouts/social_media" %>
-  </div>
-</div>
diff --git a/config/routes.rb b/config/routes.rb
index 22e3a6bdc..3398f7c6c 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -427,8 +427,12 @@ Rails.application.routes.draw do
 
     get "/page/:slug" => "pages#show"
 
+    # TODO: [forem/teamsmash] removed the /p/information view and added a redirect for SEO purposes.
+    # We need to remove this route in 2 months (11 January 2021).
+    get "/p/information", to: redirect("/about")
+
     scope "p" do
-      pages_actions = %w[welcome editor_guide publishing_from_rss_guide information markdown_basics badges].freeze
+      pages_actions = %w[welcome editor_guide publishing_from_rss_guide markdown_basics badges].freeze
       pages_actions.each do |action|
         get action, action: action, controller: "pages"
       end