From 3d0b7f10349e84912f4980e6ecac3bca6796ba7a Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 21 Apr 2017 16:40:16 +0200
Subject: [PATCH 1/5] prevents pages indexation on robots.txt

---
 public/robots.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/public/robots.txt b/public/robots.txt
index 3c9c7c01f..93780f8d1 100644
--- a/public/robots.txt
+++ b/public/robots.txt
@@ -3,3 +3,10 @@
 # To ban all spiders from the entire site uncomment the next two lines:
 # User-agent: *
 # Disallow: /
+
+Disallow: /users/
+Disallow: /comments/
+
+Disallow: /*?*locale
+Disallow: /*?*order
+Disallow: /*?*search

From 8b42d57e18ee8a1c77422e154d7cd059bea9b3ff Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 21 Apr 2017 16:59:11 +0200
Subject: [PATCH 2/5] adds rel nofollow on links

---
 app/views/devise/menu/_login_items.html.erb | 17 +++++++++++------
 app/views/layouts/_footer.html.erb          | 14 +++++++-------
 app/views/shared/_top_links.html.erb        |  3 +++
 3 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/app/views/devise/menu/_login_items.html.erb b/app/views/devise/menu/_login_items.html.erb
index b7db4a22c..5d493131d 100644
--- a/app/views/devise/menu/_login_items.html.erb
+++ b/app/views/devise/menu/_login_items.html.erb
@@ -1,6 +1,6 @@
 <% if user_signed_in? %>
   <li>
-    <%= link_to notifications_path, class: "notifications" do %>
+    <%= link_to notifications_path, rel: "nofollow", class: "notifications" do %>
       <span class="sr-only"><%= t("layouts.header.notifications") %></span>
       <% if current_user.notifications_count > 0 %>
         <span class="icon-circle" aria-hidden="true"></span>
@@ -12,19 +12,24 @@
     <% end %>
   </li>
   <li>
-    <%= link_to t("layouts.header.my_activity_link"), user_path(current_user) %>
+    <%= link_to t("layouts.header.my_activity_link"),
+                user_path(current_user), rel: "nofollow" %>
   </li>
   <li>
-    <%= link_to t("layouts.header.my_account_link"), account_path %>
+    <%= link_to t("layouts.header.my_account_link"),
+                account_path, rel: "nofollow" %>
   </li>
   <li>
-    <%= link_to t("devise_views.menu.login_items.logout"), destroy_user_session_path, method: :delete %>
+    <%= link_to t("devise_views.menu.login_items.logout"),
+                destroy_user_session_path, rel: "nofollow", method: :delete %>
   </li>
 <% else %>
   <li>
-    <%= link_to t("devise_views.menu.login_items.login"), new_user_session_path %>
+    <%= link_to t("devise_views.menu.login_items.login"),
+                new_user_session_path, rel: "nofollow" %>
   </li>
   <li>
-    <%= link_to t("devise_views.menu.login_items.signup"), new_user_registration_path, class: "button" %>
+    <%= link_to t("devise_views.menu.login_items.signup"),
+                new_user_registration_path, rel: "nofollow", class: "button" %>
   </li>
 <% end %>
diff --git a/app/views/layouts/_footer.html.erb b/app/views/layouts/_footer.html.erb
index 0bce1cd67..35e7d63a0 100644
--- a/app/views/layouts/_footer.html.erb
+++ b/app/views/layouts/_footer.html.erb
@@ -25,14 +25,14 @@
 
       <div class="small-12 medium-4 column">
         <h2>
-          <%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url") %>
+          <%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url"), rel: "nofollow" %>
         </h2>
         <p><%= t("layouts.footer.transparency_text") %></p>
       </div>
 
       <div class="small-12 medium-4 column">
         <h2>
-          <%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url") %>
+          <%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"), rel: "nofollow" %>
         </h2>
         <p><%= t("layouts.footer.open_data_text") %></p>
       </div>
@@ -54,7 +54,7 @@
         <ul>
           <% if setting['twitter_handle'] %>
             <li class="inline-block">
-              <%= link_to "https://twitter.com/#{setting['twitter_handle']}", target: "_blank", title: t("social.twitter") + t('shared.target_blank_html') do %>
+              <%= link_to "https://twitter.com/#{setting['twitter_handle']}", target: "_blank", title: t("social.twitter") + t('shared.target_blank_html'), rel: "nofollow" do %>
                 <span class="sr-only"><%= t("social.twitter") %></span>
                 <span class="icon-twitter" aria-hidden="true"></span>
               <% end %>
@@ -62,7 +62,7 @@
           <% end %>
           <% if setting['facebook_handle'] %>
             <li class="inline-block">
-              <%= link_to "https://www.facebook.com/#{setting['facebook_handle']}/", target: "_blank", title: t("social.facebook") + t('shared.target_blank_html') do %>
+              <%= link_to "https://www.facebook.com/#{setting['facebook_handle']}/", target: "_blank", title: t("social.facebook") + t('shared.target_blank_html'), rel: "nofollow" do %>
                 <span class="sr-only"><%= t("social.facebook") %></span>
                 <span class="icon-facebook" aria-hidden="true"></span>
               <% end %>
@@ -70,7 +70,7 @@
           <% end %>
           <%  if setting['blog_url'] %>
             <li class="inline-block">
-              <%= link_to setting['blog_url'], target: "_blank", title: t("social.blog") + t('shared.target_blank_html') do %>
+              <%= link_to setting['blog_url'], target: "_blank", title: t("social.blog") + t('shared.target_blank_html'), rel: "nofollow" do %>
                 <span class="sr-only"><%= t("social.blog") %></span>
                 <span class="icon-blog" aria-hidden="true"></span>
               <% end %>
@@ -78,7 +78,7 @@
           <% end %>
           <% if setting['youtube_handle'] %>
             <li class="inline-block">
-              <%= link_to "https://www.youtube.com/#{setting['youtube_handle']}", target: "_blank", title: t("social.youtube") + t('shared.target_blank_html') do %>
+              <%= link_to "https://www.youtube.com/#{setting['youtube_handle']}", target: "_blank", title: t("social.youtube") + t('shared.target_blank_html'), rel: "nofollow" do %>
                 <span class="sr-only"><%= t("social.youtube") %></span>
                 <span class="icon-youtube" aria-hidden="true"></span>
               <% end %>
@@ -86,7 +86,7 @@
           <% end %>
           <% if setting['telegram_handle'] %>
             <li class="inline-block">
-              <%= link_to "https://www.telegram.me/#{setting['telegram_handle']}", target: "_blank", title: t("social.telegram") + t('shared.target_blank_html') do %>
+              <%= link_to "https://www.telegram.me/#{setting['telegram_handle']}", target: "_blank", title: t("social.telegram") + t('shared.target_blank_html'), rel: "nofollow" do %>
                 <span class="sr-only"><%= t("social.telegram") %></span>
                 <span class="icon-telegram" aria-hidden="true"></span>
               <% end %>
diff --git a/app/views/shared/_top_links.html.erb b/app/views/shared/_top_links.html.erb
index 8a71130d6..ae77cb1c2 100644
--- a/app/views/shared/_top_links.html.erb
+++ b/app/views/shared/_top_links.html.erb
@@ -3,12 +3,14 @@
     <%= link_to t("layouts.header.external_link_transparency"),
                 setting['transparency_url'].presence || t("layouts.header.external_link_transparency_url"),
                 target: "_blank",
+                rel: "nofollow",
                 title: t('shared.target_blank_html') %>
   </li>
   <li>
     <%= link_to t("layouts.header.external_link_opendata"),
                 setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"),
                 target: "_blank",
+                rel: "nofollow",
                 title: t('shared.target_blank_html') %>
   </li>
 
@@ -17,6 +19,7 @@
       <%= link_to t("layouts.header.external_link_blog"),
                   setting['blog_url'],
                   target: "_blank",
+                  rel: "nofollow",
                   title: t('shared.target_blank_html') %>
     </li>
   <% end %>

From 4a06e34aa6978989f714bd5ad519e0c4ace13dda Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 21 Apr 2017 16:59:28 +0200
Subject: [PATCH 3/5] fixes old faq page path

---
 app/views/layouts/_footer.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/layouts/_footer.html.erb b/app/views/layouts/_footer.html.erb
index 35e7d63a0..eedd0ac89 100644
--- a/app/views/layouts/_footer.html.erb
+++ b/app/views/layouts/_footer.html.erb
@@ -11,7 +11,7 @@
           consul:  link_to(t("layouts.footer.consul"), t("layouts.footer.consul_url"), target: "blank")).html_safe
         %>
         <%= t("layouts.footer.contact_us") %>
-        <%= link_to t("layouts.footer.faq"), page_path('faq') %>
+        <%= link_to t("layouts.footer.faq"), faq_path %>
       </p>
     </div>
 

From 05c366e2fff98f04a3f8ea303f33696e1451ca87 Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 21 Apr 2017 17:10:11 +0200
Subject: [PATCH 4/5] removes headings on footer

---
 app/assets/stylesheets/layout.scss |  8 ++------
 app/views/layouts/_footer.html.erb | 12 +++---------
 2 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/app/assets/stylesheets/layout.scss b/app/assets/stylesheets/layout.scss
index 94d403528..7c204bced 100644
--- a/app/assets/stylesheets/layout.scss
+++ b/app/assets/stylesheets/layout.scss
@@ -587,12 +587,8 @@ footer {
     padding-left: 0;
   }
 
-  h2 a {
-    border-bottom: 1px solid $text-light;
-    display: block;
-    font-size: rem-calc(24);
-    line-height: rem-calc(31);
-    padding-bottom: $line-height/4;
+  a.title {
+    font-weight: bold;
     text-decoration: none;
   }
 }
diff --git a/app/views/layouts/_footer.html.erb b/app/views/layouts/_footer.html.erb
index eedd0ac89..2e2f5c063 100644
--- a/app/views/layouts/_footer.html.erb
+++ b/app/views/layouts/_footer.html.erb
@@ -17,23 +17,17 @@
 
     <div class="footer-sections small-12 large-8 column">
       <div class="small-12 medium-4 column">
-        <h2>
-          <%= link_to t("layouts.footer.participation_title"), root_path %>
-        </h2>
+        <%= link_to t("layouts.footer.participation_title"), root_path, class: "title" %>
         <p><%= t("layouts.footer.participation_text") %></p>
       </div>
 
       <div class="small-12 medium-4 column">
-        <h2>
-          <%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url"), rel: "nofollow" %>
-        </h2>
+        <%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url"), class: "title", rel: "nofollow" %>
         <p><%= t("layouts.footer.transparency_text") %></p>
       </div>
 
       <div class="small-12 medium-4 column">
-        <h2>
-          <%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"), rel: "nofollow" %>
-        </h2>
+        <%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"), class: "title", rel: "nofollow" %>
         <p><%= t("layouts.footer.open_data_text") %></p>
       </div>
     </div>

From f7020618da26ef20fd35301b5d271d4e9a1772f7 Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 21 Apr 2017 20:08:46 +0200
Subject: [PATCH 5/5] adds pages, budgets and polls to sitemap.rb

---
 config/sitemap.rb | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/config/sitemap.rb b/config/sitemap.rb
index 11ec9fb09..e5537dd9f 100644
--- a/config/sitemap.rb
+++ b/config/sitemap.rb
@@ -15,7 +15,11 @@ SitemapGenerator::Sitemap.create do
     add page_name if page_name.present?
   end
 
-  add "help_translate"
+  add more_info_path
+  add how_to_use_path
+  add faq_path
+  add participation_facts_path
+  add participation_world_path
 
   add debates_path, priority: 0.7, changefreq: "daily"
   Debate.find_each do |debate|
@@ -27,10 +31,18 @@ SitemapGenerator::Sitemap.create do
     add proposal_path(proposal), lastmod: proposal.updated_at
   end
 
-  add proposal_ballots_path
-
   add spending_proposals_path, priority: 0.7, changefreq: "daily"
   SpendingProposal.find_each do |spending_proposal|
     add spending_proposal_path(spending_proposal), lastmod: spending_proposal.updated_at
   end
+
+  add budgets_path, priority: 0.7, changefreq: "daily"
+  Budget.find_each do |budget|
+    add budget_path(budget), lastmod: budget.updated_at
+  end
+
+  add polls_path, priority: 0.7, changefreq: "daily"
+  Poll.find_each do |poll|
+    add poll_path(poll), lastmod: poll.starts_at
+  end
 end