From 3fd97b0a9b313dbe3c668008f89e30192d0ab3d1 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Thu, 12 Apr 2018 11:21:15 +0200
Subject: [PATCH 1/5] Rename Newsletter & Notification to User Messages

We're increasing the features around messaging users, menu name has to
be generic enough.
---
 app/views/admin/_menu.html.erb | 16 ++++++++--------
 config/locales/en/admin.yml    |  2 +-
 config/locales/es/admin.yml    |  2 +-
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/views/admin/_menu.html.erb b/app/views/admin/_menu.html.erb
index e5af32cc4..e1d6cc0de 100644
--- a/app/views/admin/_menu.html.erb
+++ b/app/views/admin/_menu.html.erb
@@ -79,21 +79,21 @@
       </li>
     <% end %>
 
-    <% newsletters_notifications_sections = %w(newsletters emails_download admin_notifications) %>
-    <% newsletters_menu_active = newsletters_notifications_sections.include?(controller_name) %>
-    <li class="section-title" <%= "class=active" if newsletters_menu_active %>>
+    <% messages_sections = %w(newsletters emails_download admin_notifications) %>
+    <% messages_menu_active = messages_sections.include?(controller_name) %>
+    <li class="section-title" <%= "class=is-active" if messages_menu_active %>>
       <a href="#">
         <span class="icon-zip"></span>
-        <strong><%= t("admin.menu.newsletters_and_notifications") %></strong>
+        <strong><%= t("admin.menu.messaging_users") %></strong>
       </a>
-      <ul id="newsletters_and_notifications_menu" <%= "class=is-active" if newsletters_menu_active %>>
-        <li <%= "class=active" if controller_name == "newsletters" %>>
+      <ul id="messaging_users_menu" <%= "class=is-active" if messages_menu_active %>>
+        <li <%= "class=is-active" if controller_name == "newsletters" %>>
           <%= link_to t("admin.menu.newsletters"), admin_newsletters_path %>
         </li>
-        <li <%= "class=active" if controller_name == "admin_notifications" %>>
+        <li <%= "class=is-active" if controller_name == "admin_notifications" %>>
           <%= link_to t("admin.menu.admin_notifications"), admin_admin_notifications_path %>
         </li>
-        <li <%= "class=active" if controller_name == "emails_download" %>>
+        <li <%= "class=is-active" if controller_name == "emails_download" %>>
           <%= link_to t("admin.menu.emails_download"), admin_emails_download_index_path %>
         </li>
       </ul>
diff --git a/config/locales/en/admin.yml b/config/locales/en/admin.yml
index 54bb5adc6..43f91eda1 100644
--- a/config/locales/en/admin.yml
+++ b/config/locales/en/admin.yml
@@ -518,7 +518,7 @@ en:
       administrators: Administrators
       managers: Managers
       moderators: Moderators
-      newsletters_and_notifications: Newsletters & Notifications
+      messaging_users: Messages to users
       newsletters: Newsletters
       admin_notifications: Notifications
       emails_download: Emails download
diff --git a/config/locales/es/admin.yml b/config/locales/es/admin.yml
index c57b42197..4c261ae1f 100644
--- a/config/locales/es/admin.yml
+++ b/config/locales/es/admin.yml
@@ -519,7 +519,7 @@ es:
       administrators: Administradores
       managers: Gestores
       moderators: Moderadores
-      newsletters_and_notifications: Newsletters & Notificaciones
+      messaging_users: Mensajes a usuarios
       newsletters: Newsletters
       admin_notifications: Notificaciones
       emails_download: Descarga de emails

From b9c6da245a0b79aa63be745db0392e5a43ef7993 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Thu, 12 Apr 2018 19:13:49 +0200
Subject: [PATCH 2/5] Add System emails menu & list to admin panel

We need a section to list all System Emails to view the templates and
preview an example of what would be sent.
---
 .../admin/system_emails_controller.rb         |  7 +++++
 app/views/admin/_menu.html.erb                |  5 +++-
 app/views/admin/system_emails/index.html.erb  | 29 +++++++++++++++++++
 config/locales/en/admin.yml                   |  7 +++++
 config/locales/es/admin.yml                   |  7 +++++
 config/routes/admin.rb                        |  5 ++++
 6 files changed, 59 insertions(+), 1 deletion(-)
 create mode 100644 app/controllers/admin/system_emails_controller.rb
 create mode 100644 app/views/admin/system_emails/index.html.erb

diff --git a/app/controllers/admin/system_emails_controller.rb b/app/controllers/admin/system_emails_controller.rb
new file mode 100644
index 000000000..de09c7c1f
--- /dev/null
+++ b/app/controllers/admin/system_emails_controller.rb
@@ -0,0 +1,7 @@
+class Admin::SystemEmailsController < Admin::BaseController
+
+  def index
+    @system_emails = %w(proposal_notification_digest)
+  end
+
+end
diff --git a/app/views/admin/_menu.html.erb b/app/views/admin/_menu.html.erb
index e1d6cc0de..ffec03232 100644
--- a/app/views/admin/_menu.html.erb
+++ b/app/views/admin/_menu.html.erb
@@ -79,7 +79,7 @@
       </li>
     <% end %>
 
-    <% messages_sections = %w(newsletters emails_download admin_notifications) %>
+    <% messages_sections = %w(newsletters emails_download admin_notifications system_emails) %>
     <% messages_menu_active = messages_sections.include?(controller_name) %>
     <li class="section-title" <%= "class=is-active" if messages_menu_active %>>
       <a href="#">
@@ -93,6 +93,9 @@
         <li <%= "class=is-active" if controller_name == "admin_notifications" %>>
           <%= link_to t("admin.menu.admin_notifications"), admin_admin_notifications_path %>
         </li>
+        <li <%= "class=is-active" if controller_name == "system_emails" %>>
+          <%= link_to t("admin.menu.system_emails"), admin_system_emails_path %>
+        </li>
         <li <%= "class=is-active" if controller_name == "emails_download" %>>
           <%= link_to t("admin.menu.emails_download"), admin_emails_download_index_path %>
         </li>
diff --git a/app/views/admin/system_emails/index.html.erb b/app/views/admin/system_emails/index.html.erb
new file mode 100644
index 000000000..d2962fba2
--- /dev/null
+++ b/app/views/admin/system_emails/index.html.erb
@@ -0,0 +1,29 @@
+<h2 class="inline-block"><%= t("admin.menu.system_emails") %></h2>
+
+<table id="system_emails">
+  <thead>
+    <tr>
+      <th><%= t("admin.shared.title") %></th>
+      <th><%= t("admin.shared.description") %></th>
+      <th class="small-5 text-right"><%= t("admin.shared.actions") %></th>
+    </tr>
+  </thead>
+  <tbody>
+  <% @system_emails.each do |system_email| %>
+    <tr id="<%= system_email %>" class="system_email">
+      <td>
+        <%= t("admin.system_emails.#{system_email}.title") %>
+      </td>
+      <td>
+        <%= t("admin.system_emails.#{system_email}.description") %>
+      </td>
+      <td class="text-right">
+        <%= link_to t("admin.shared.view"), admin_system_email_view_path(system_email),
+                                            class: "button hollow" %>
+        <%= link_to t("admin.shared.preview"), admin_system_email_preview_path(system_email),
+                                               class: "button" %>
+      </td>
+    </tr>
+  <% end %>
+  </tbody>
+</table>
diff --git a/config/locales/en/admin.yml b/config/locales/en/admin.yml
index 43f91eda1..ea01f35d6 100644
--- a/config/locales/en/admin.yml
+++ b/config/locales/en/admin.yml
@@ -521,6 +521,7 @@ en:
       messaging_users: Messages to users
       newsletters: Newsletters
       admin_notifications: Notifications
+      system_emails: System Emails
       emails_download: Emails download
       valuators: Valuators
       poll_officers: Poll officers
@@ -649,6 +650,10 @@ en:
         preview_guide: "This is how the users will see the notification:"
         sent_guide: "This is how the users see the notification:"
         send_alert: Are you sure you want to send this notification to %{n} users?
+    system_emails:
+      proposal_notification_digest:
+        title: Proposal Notification Digest
+        description: Gathers all proposal notifications for an user in a single message, to avoid too much emails.
     emails_download:
       index:
         title: Emails download
@@ -1045,6 +1050,8 @@ en:
       image: Image
       show_image: Show image
       moderated_content: "Check the content moderated by the moderators, and confirm if the moderation has been done correctly."
+      view: View
+      preview: Preview
     spending_proposals:
       index:
         geozone_filter_all: All zones
diff --git a/config/locales/es/admin.yml b/config/locales/es/admin.yml
index 4c261ae1f..62e806ffb 100644
--- a/config/locales/es/admin.yml
+++ b/config/locales/es/admin.yml
@@ -522,6 +522,7 @@ es:
       messaging_users: Mensajes a usuarios
       newsletters: Newsletters
       admin_notifications: Notificaciones
+      system_emails: Emails del sistema
       emails_download: Descarga de emails
       valuators: Evaluadores
       poll_officers: Presidentes de mesa
@@ -650,6 +651,10 @@ es:
         preview_guide: "Así es como los usuarios verán la notificación:"
         sent_guide: "Así es como los usuarios ven la notificación:"
         send_alert: ¿Estás seguro/a de que quieres enviar esta notificación a %{n} usuarios?
+    system_emails:
+      proposal_notification_digest:
+        title: Resumen de Notificationes de Propuestas
+        description: Reune todas las notificaciones de propuestas en un único mensaje, para evitar demasiados emails.
     emails_download:
       index:
         title: Descarga de emails
@@ -1046,6 +1051,8 @@ es:
       image: Imagen
       show_image: Mostrar imagen
       moderated_content: "Revisa el contenido moderado por los moderadores, y confirma si la moderación se ha realizado correctamente."
+      view: Ver
+      preview: Previsualizar
     spending_proposals:
       index:
         geozone_filter_all: Todos los ámbitos de actuación
diff --git a/config/routes/admin.rb b/config/routes/admin.rb
index ec7e4bb5f..7652ecefc 100644
--- a/config/routes/admin.rb
+++ b/config/routes/admin.rb
@@ -165,6 +165,11 @@ namespace :admin do
     end
   end
 
+  resources :system_emails, only: [:index] do
+    get :view
+    get :preview
+  end
+
   resources :emails_download, only: :index do
     get :generate_csv, on: :collection
   end

From 5c726e0949297b451481701bb4bbede886824154 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Thu, 12 Apr 2018 19:17:33 +0200
Subject: [PATCH 3/5] Add Admin System Emails view & first system email

Admins need to be able to see what a particular System Email looks like
with dummy data.

Also adding the first system email to be managed: the proposal
notification digest.

In case the current admin doesn't have any Notifications from
 ProposalNotifications there will be a crash. We'll solve this in later
 PR's with a system to "inject" sample text in email templates without
 having to generate records in the database.
---
 .../admin/system_emails_controller.rb         | 15 +++++++
 app/views/admin/system_emails/index.html.erb  |  2 -
 app/views/admin/system_emails/view.html.erb   | 36 ++++++++++++++++
 config/locales/en/seeds.yml                   | 15 -------
 config/locales/es/seeds.yml                   | 15 -------
 config/routes/admin.rb                        |  1 -
 spec/features/admin/system_emails_spec.rb     | 43 +++++++++++++++++++
 7 files changed, 94 insertions(+), 33 deletions(-)
 create mode 100644 app/views/admin/system_emails/view.html.erb
 create mode 100644 spec/features/admin/system_emails_spec.rb

diff --git a/app/controllers/admin/system_emails_controller.rb b/app/controllers/admin/system_emails_controller.rb
index de09c7c1f..da86b1e9a 100644
--- a/app/controllers/admin/system_emails_controller.rb
+++ b/app/controllers/admin/system_emails_controller.rb
@@ -1,7 +1,22 @@
 class Admin::SystemEmailsController < Admin::BaseController
 
+  before_action :load_system_email, only: [:view]
+
   def index
     @system_emails = %w(proposal_notification_digest)
   end
 
+  def view
+    case @system_email
+    when "proposal_notification_digest"
+      @notifications = Notification.where(notifiable_type: "ProposalNotification").limit(2)
+      @subject = t('mailers.proposal_notification_digest.title', org_name: Setting['org_name'])
+    end
+  end
+
+  private
+
+  def load_system_email
+    @system_email = params[:system_email_id]
+  end
 end
diff --git a/app/views/admin/system_emails/index.html.erb b/app/views/admin/system_emails/index.html.erb
index d2962fba2..6e294d4e5 100644
--- a/app/views/admin/system_emails/index.html.erb
+++ b/app/views/admin/system_emails/index.html.erb
@@ -20,8 +20,6 @@
       <td class="text-right">
         <%= link_to t("admin.shared.view"), admin_system_email_view_path(system_email),
                                             class: "button hollow" %>
-        <%= link_to t("admin.shared.preview"), admin_system_email_preview_path(system_email),
-                                               class: "button" %>
       </td>
     </tr>
   <% end %>
diff --git a/app/views/admin/system_emails/view.html.erb b/app/views/admin/system_emails/view.html.erb
new file mode 100644
index 000000000..9aaeb5226
--- /dev/null
+++ b/app/views/admin/system_emails/view.html.erb
@@ -0,0 +1,36 @@
+<%= back_link_to admin_system_emails_path %>
+
+<h2><%= t("admin.system_emails.#{@system_email}.title") %></h2>
+
+<div class="small-12 column">
+  <div class="callout highlight">
+    <div class="row">
+      <div class="small-12 medium-4 column">
+        <strong><%= t("admin.newsletters.show.from") %></strong><br>
+        <%= Setting['mailer_from_address'] %>
+      </div>
+      <div class="small-12 medium-8 column">
+        <strong><%= t("admin.newsletters.show.subject") %></strong><br>
+        <%= @subject %>
+      </div>
+    </div>
+  </div>
+
+  <strong><%= t("admin.newsletters.show.body") %></strong>
+  <p class="help-text" id="phase-description-help-text">
+    <%= t("admin.newsletters.show.body_help_text") %>
+  </p>
+  <div class="newsletter-body-content">
+    <%= render file: "app/views/layouts/_mailer_header.html.erb" %>
+
+    <table cellpadding="0" cellspacing="0" border="0" style="background: #fff; margin: 0 auto; max-width: 700px; width:100%;">
+      <tbody>
+        <tr>
+          <%= render file: "app/views/mailer/#{@system_email}.html.erb" %>
+        </tr>
+      </tbody>
+    </table>
+
+    <%= render file: "app/views/layouts/_mailer_footer.html.erb" %>
+  </div>
+</div>
diff --git a/config/locales/en/seeds.yml b/config/locales/en/seeds.yml
index a3587c9e3..4e40cfd63 100644
--- a/config/locales/en/seeds.yml
+++ b/config/locales/en/seeds.yml
@@ -53,18 +53,3 @@ en:
       recounting_poll: "Recounting Poll"
       expired_poll_without_stats: "Expired Poll without Stats & Results"
       expired_poll_with_stats: "Expired Poll with Stats & Results"
-    admin_notifications:
-      internal_link:
-        title: 'Do you have a proposal?'
-        body: 'Remember you can create a proposal with your ideas and people will discuss & support it.'
-        link: '/proposals'
-      external_link:
-        title: Help us translate consul
-        body: 'If you are proficient in a language, please help us translate consul!.'
-        link: 'https://crwd.in/consul'
-      without_link:
-        title: 'You can now geolocate proposals & investments'
-        body: 'When you create a proposal or investment you now can specify a point on a map'
-      not_sent:
-        title: 'We are closing the Participatory Budget!!'
-        body: 'Hurry up and create a last proposal before it ends next in few days!'
diff --git a/config/locales/es/seeds.yml b/config/locales/es/seeds.yml
index d8a40471b..1d6563bef 100644
--- a/config/locales/es/seeds.yml
+++ b/config/locales/es/seeds.yml
@@ -53,18 +53,3 @@ es:
       recounting_poll: "Votación en Recuento"
       expired_poll_without_stats: "Votación Finalizada (sin Estadísticas o Resultados)"
       expired_poll_with_stats: "Votación Finalizada (con Estadísticas y Resultado)"
-    admin_notifications:
-      internal_link:
-        title: 'Tienes una propuesta?'
-        body: 'Recuerda que puedes crear propuestas y los ciudadanos las debatirán y apoyarán.'
-        link: '/proposals'
-      external_link:
-        title: 'Ayúdanos a traducir CONSUL'
-        body: 'Si dominas un idioma, ayúdanos a completar su traducción en CONSUL.'
-        link: 'https://crwd.in/consul'
-      without_link:
-        title: 'Ahora puedes geolocalizar propuestas y proyectos de inversión'
-        body: 'Cuando crees una propuesta o proyecto de inversión podrás especificar su localización en el mapa'
-      not_sent:
-        title: 'Últimos días para crear proyectos de Presupuestos Participativos'
-        body: 'Quedan pocos dias para que se cierre el plazo de presentación de proyectos de inversión para los presupuestos participativos!'
diff --git a/config/routes/admin.rb b/config/routes/admin.rb
index 7652ecefc..2fe805593 100644
--- a/config/routes/admin.rb
+++ b/config/routes/admin.rb
@@ -167,7 +167,6 @@ namespace :admin do
 
   resources :system_emails, only: [:index] do
     get :view
-    get :preview
   end
 
   resources :emails_download, only: :index do
diff --git a/spec/features/admin/system_emails_spec.rb b/spec/features/admin/system_emails_spec.rb
new file mode 100644
index 000000000..ddd812236
--- /dev/null
+++ b/spec/features/admin/system_emails_spec.rb
@@ -0,0 +1,43 @@
+require 'rails_helper'
+
+feature "System Emails" do
+
+  background do
+    admin = create(:administrator)
+    login_as(admin.user)
+  end
+
+  context "Index" do
+    scenario "Lists all system emails with correct actions" do
+      visit admin_system_emails_path
+
+      within('#proposal_notification_digest') do
+        expect(page).to have_link('View')
+      end
+    end
+  end
+
+  context "View" do
+    scenario "#proposal_notification_digest" do
+      proposal_a = create(:proposal, title: 'Proposal A')
+      proposal_b = create(:proposal, title: 'Proposal B')
+      proposal_notification_a = create(:proposal_notification, proposal: proposal_a,
+                                                               title: 'Proposal A Title',
+                                                               body: 'Proposal A Notification Body')
+      proposal_notification_b = create(:proposal_notification, proposal: proposal_b,
+                                                               title: 'Proposal B Title',
+                                                               body: 'Proposal B Notification Body')
+      notification_a = create(:notification, notifiable: proposal_notification_a)
+      notification_b = create(:notification, notifiable: proposal_notification_b)
+
+      visit admin_system_email_view_path('proposal_notification_digest')
+
+      expect(page).to have_content('Proposal notifications in')
+      expect(page).to have_link('Proposal A Title', href: notification_url(notification_a))
+      expect(page).to have_link('Proposal B Title', href: notification_url(notification_b))
+      expect(page).to have_content('Proposal A Notification Body')
+      expect(page).to have_content('Proposal B Notification Body')
+    end
+  end
+
+end

From eb0919d978ca22a5fe6dcdeaf49d37c2a293786e Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Wed, 18 Apr 2018 16:12:38 +0200
Subject: [PATCH 4/5] Add Preview Pending action to Admin System Emails

Some system emails need to be manually reviewed before being sent. This
new action allows admins to see a preview of all Pending to be sent
Proposal Notification Digest messages.
---
 .../admin/system_emails_controller.rb         | 19 +++++++++--
 app/views/admin/system_emails/index.html.erb  | 19 +++++++----
 .../system_emails/preview_pending.html.erb    | 16 +++++++++
 .../_proposal_notification_digest.html.erb    | 33 +++++++++++++++++++
 config/locales/en/admin.yml                   | 10 +++++-
 config/locales/es/admin.yml                   | 10 +++++-
 config/routes/admin.rb                        |  1 +
 spec/features/admin/system_emails_spec.rb     | 25 ++++++++++++++
 8 files changed, 123 insertions(+), 10 deletions(-)
 create mode 100644 app/views/admin/system_emails/preview_pending.html.erb
 create mode 100644 app/views/admin/system_emails/preview_pending/_proposal_notification_digest.html.erb

diff --git a/app/controllers/admin/system_emails_controller.rb b/app/controllers/admin/system_emails_controller.rb
index da86b1e9a..626680f43 100644
--- a/app/controllers/admin/system_emails_controller.rb
+++ b/app/controllers/admin/system_emails_controller.rb
@@ -1,9 +1,11 @@
 class Admin::SystemEmailsController < Admin::BaseController
 
-  before_action :load_system_email, only: [:view]
+  before_action :load_system_email, only: [:view, :preview_pending]
 
   def index
-    @system_emails = %w(proposal_notification_digest)
+    @system_emails = {
+      proposal_notification_digest: %w(view preview_pending)
+    }
   end
 
   def view
@@ -14,9 +16,22 @@ class Admin::SystemEmailsController < Admin::BaseController
     end
   end
 
+  def preview_pending
+    case @system_email
+    when "proposal_notification_digest"
+      @previews = ProposalNotification.where(id: unsent_proposal_notifications_ids)
+                                      .page(params[:page])
+    end
+  end
+
   private
 
   def load_system_email
     @system_email = params[:system_email_id]
   end
+
+  def unsent_proposal_notifications_ids
+    Notification.where(notifiable_type: "ProposalNotification", emailed_at: nil)
+                .group(:notifiable_id).count.keys
+  end
 end
diff --git a/app/views/admin/system_emails/index.html.erb b/app/views/admin/system_emails/index.html.erb
index 6e294d4e5..4d135d615 100644
--- a/app/views/admin/system_emails/index.html.erb
+++ b/app/views/admin/system_emails/index.html.erb
@@ -9,17 +9,24 @@
     </tr>
   </thead>
   <tbody>
-  <% @system_emails.each do |system_email| %>
-    <tr id="<%= system_email %>" class="system_email">
+  <% @system_emails.each do |system_email_title, system_email_actions| %>
+    <tr id="<%= system_email_title %>" class="system_email">
       <td>
-        <%= t("admin.system_emails.#{system_email}.title") %>
+        <%= t("admin.system_emails.#{system_email_title}.title") %>
       </td>
       <td>
-        <%= t("admin.system_emails.#{system_email}.description") %>
+        <%= t("admin.system_emails.#{system_email_title}.description") %>
       </td>
       <td class="text-right">
-        <%= link_to t("admin.shared.view"), admin_system_email_view_path(system_email),
-                                            class: "button hollow" %>
+        <% if system_email_actions.include?('view') %>
+          <%= link_to t("admin.shared.view"), admin_system_email_view_path(system_email_title),
+                                              class: "button hollow" %>
+        <% end %>
+        <% if system_email_actions.include?('preview_pending') %>
+          <%= link_to t("admin.system_emails.preview_pending.action"),
+                      admin_system_email_preview_pending_path(system_email_title),
+                      class: "button" %>
+        <% end %>
       </td>
     </tr>
   <% end %>
diff --git a/app/views/admin/system_emails/preview_pending.html.erb b/app/views/admin/system_emails/preview_pending.html.erb
new file mode 100644
index 000000000..6de1c993f
--- /dev/null
+++ b/app/views/admin/system_emails/preview_pending.html.erb
@@ -0,0 +1,16 @@
+<%= back_link_to admin_system_emails_path %>
+
+<% system_email_name = t("admin.system_emails.#{@system_email}.title") %>
+<h2><%= t("admin.system_emails.preview_pending.preview_of", name: system_email_name) %></h2>
+
+<div class="small-12 column">
+  <h4><%= t("admin.system_emails.preview_pending.pending_to_be_sent") %></h4>
+  <p><%= t("admin.system_emails.#{@system_email}.preview_detail") %></p>
+
+  <% @previews.each do |preview| %>
+    <%= render partial: "admin/system_emails/preview_pending/#{@system_email}",
+               locals: { preview: preview } %>
+  <% end %>
+</div>
+
+<%= paginate @previews %>
diff --git a/app/views/admin/system_emails/preview_pending/_proposal_notification_digest.html.erb b/app/views/admin/system_emails/preview_pending/_proposal_notification_digest.html.erb
new file mode 100644
index 000000000..7cdd4d66d
--- /dev/null
+++ b/app/views/admin/system_emails/preview_pending/_proposal_notification_digest.html.erb
@@ -0,0 +1,33 @@
+<% if preview.proposal.present? %>
+  <div class="callout highlight">
+    <div class="row">
+      <div class="small-12 medium-6 column">
+        <strong><%= t("admin.shared.proposal") %></strong><br>
+        <%= link_to preview.proposal.title, proposal_url(preview.proposal) %>
+      </div>
+      <div class="small-12 medium-6 column">
+        <strong><%= t("admin.shared.title") %></strong><br>
+        <%= link_to preview.title, proposal_url(preview.proposal, anchor: "tab-notifications") %>
+      </div>
+    </div>
+    <div class="row">
+      <div class="small-12 medium-6 column">
+        <strong><%= t("admin.shared.author") %></strong><br>
+        <%= preview.proposal.author&.username || '-' %>
+      </div>
+      <div class="small-12 medium-6 column">
+        <strong><%= t("admin.shared.created_at") %></strong><br>
+        <%= l(preview.created_at, format: :datetime) %>
+      </div>
+    </div>
+  </div>
+
+  <div class="row">
+    <div class="column">
+      <strong><%= t("admin.shared.content") %></strong>
+      <p class="help-text" id="phase-description-help-text">
+        <%= preview.body %>
+      </p>
+    </div>
+  </div>
+<% end %>
diff --git a/config/locales/en/admin.yml b/config/locales/en/admin.yml
index ea01f35d6..a2194110b 100644
--- a/config/locales/en/admin.yml
+++ b/config/locales/en/admin.yml
@@ -651,9 +651,14 @@ en:
         sent_guide: "This is how the users see the notification:"
         send_alert: Are you sure you want to send this notification to %{n} users?
     system_emails:
+      preview_pending:
+        action: Preview Pending
+        preview_of: Preview of %{name}
+        pending_to_be_sent: This is the content pending to be sent
       proposal_notification_digest:
         title: Proposal Notification Digest
         description: Gathers all proposal notifications for an user in a single message, to avoid too much emails.
+        preview_detail: Users will only recieve notifications from the proposals they are following
     emails_download:
       index:
         title: Emails download
@@ -1051,7 +1056,10 @@ en:
       show_image: Show image
       moderated_content: "Check the content moderated by the moderators, and confirm if the moderation has been done correctly."
       view: View
-      preview: Preview
+      proposal: Proposal
+      author: Author
+      content: Content
+      created_at: Created at
     spending_proposals:
       index:
         geozone_filter_all: All zones
diff --git a/config/locales/es/admin.yml b/config/locales/es/admin.yml
index 62e806ffb..2d652e8e2 100644
--- a/config/locales/es/admin.yml
+++ b/config/locales/es/admin.yml
@@ -652,9 +652,14 @@ es:
         sent_guide: "Así es como los usuarios ven la notificación:"
         send_alert: ¿Estás seguro/a de que quieres enviar esta notificación a %{n} usuarios?
     system_emails:
+      preview_pending:
+        action: Previsualizar Pendientes
+        preview_of: Vista previa de %{name}
+        pending_to_be_sent: Este es todo el contenido pendiente de enviar
       proposal_notification_digest:
         title: Resumen de Notificationes de Propuestas
         description: Reune todas las notificaciones de propuestas en un único mensaje, para evitar demasiados emails.
+        preview_detail: Los usuarios sólo recibirán las notificaciones de aquellas propuestas que siguen.
     emails_download:
       index:
         title: Descarga de emails
@@ -1052,7 +1057,10 @@ es:
       show_image: Mostrar imagen
       moderated_content: "Revisa el contenido moderado por los moderadores, y confirma si la moderación se ha realizado correctamente."
       view: Ver
-      preview: Previsualizar
+      proposal: Propuesta
+      author: Autor
+      content: Contenido
+      created_at: Fecha de creación
     spending_proposals:
       index:
         geozone_filter_all: Todos los ámbitos de actuación
diff --git a/config/routes/admin.rb b/config/routes/admin.rb
index 2fe805593..150d1b28a 100644
--- a/config/routes/admin.rb
+++ b/config/routes/admin.rb
@@ -167,6 +167,7 @@ namespace :admin do
 
   resources :system_emails, only: [:index] do
     get :view
+    get :preview_pending
   end
 
   resources :emails_download, only: :index do
diff --git a/spec/features/admin/system_emails_spec.rb b/spec/features/admin/system_emails_spec.rb
index ddd812236..634a32efb 100644
--- a/spec/features/admin/system_emails_spec.rb
+++ b/spec/features/admin/system_emails_spec.rb
@@ -40,4 +40,29 @@ feature "System Emails" do
     end
   end
 
+  context "Preview Pending" do
+    scenario "#proposal_notification_digest" do
+      proposal_a = create(:proposal, title: 'Proposal A')
+      proposal_b = create(:proposal, title: 'Proposal B')
+      proposal_notification_a = create(:proposal_notification, proposal: proposal_a,
+                                                               title: 'Proposal A Title',
+                                                               body: 'Proposal A Notification Body')
+      proposal_notification_b = create(:proposal_notification, proposal: proposal_b,
+                                                               title: 'Proposal B Title',
+                                                               body: 'Proposal B Notification Body')
+      create(:notification, notifiable: proposal_notification_a, emailed_at: nil)
+      create(:notification, notifiable: proposal_notification_b, emailed_at: nil)
+
+      visit admin_system_email_preview_pending_path('proposal_notification_digest')
+
+      expect(page).to have_content('This is the content pending to be sent')
+      expect(page).to have_link('Proposal A', href: proposal_url(proposal_a))
+      expect(page).to have_link('Proposal B', href: proposal_url(proposal_b))
+      expect(page).to have_link('Proposal A Title', href: proposal_url(proposal_a,
+                                                    anchor: 'tab-notifications'))
+      expect(page).to have_link('Proposal B Title', href: proposal_url(proposal_b,
+                                                    anchor: 'tab-notifications'))
+    end
+  end
+
 end

From ac9f85e2083cac6a4c35eaec3340c83a383f9e69 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Wed, 18 Apr 2018 16:23:19 +0200
Subject: [PATCH 5/5] Change Proposal Notification Digest link to anchor

On the proposal notification digest messages there was a link to each
notification. Instead we need that link to point towards the notifications
tab of the proposal.
---
 app/views/mailer/proposal_notification_digest.html.erb |  2 +-
 spec/features/admin/system_emails_spec.rb              | 10 ++++++----
 spec/features/emails_spec.rb                           |  4 ++--
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/app/views/mailer/proposal_notification_digest.html.erb b/app/views/mailer/proposal_notification_digest.html.erb
index 1a0ef1468..8f80c3e0c 100644
--- a/app/views/mailer/proposal_notification_digest.html.erb
+++ b/app/views/mailer/proposal_notification_digest.html.erb
@@ -22,7 +22,7 @@
         <tr>
           <td style="padding-bottom: 20px; padding-left: 10px;">
             <p style="font-family: 'Open Sans','Helvetica Neue',arial,sans-serif;font-size: 20px;font-weight: bold;line-height: 24px; margin: 0;">
-              <%= link_to notification.notifiable.title, notification_url(notification), style: "color: #2895F1; text-decoration: none;" %>
+              <%= link_to notification.notifiable.title, proposal_url(notification.notifiable.proposal, anchor: "tab-notifications"), style: "color: #2895F1; text-decoration: none;" %>
             </p>
             <p style="font-family: 'Open Sans','Helvetica Neue',arial,sans-serif;font-size: 14px;font-weight: normal;line-height: 24px; margin-top: 0; color: #cccccc;">
               <%= notification.notifiable.proposal.title %>&nbsp;&bull;&nbsp;
diff --git a/spec/features/admin/system_emails_spec.rb b/spec/features/admin/system_emails_spec.rb
index 634a32efb..dd14e87bc 100644
--- a/spec/features/admin/system_emails_spec.rb
+++ b/spec/features/admin/system_emails_spec.rb
@@ -27,14 +27,16 @@ feature "System Emails" do
       proposal_notification_b = create(:proposal_notification, proposal: proposal_b,
                                                                title: 'Proposal B Title',
                                                                body: 'Proposal B Notification Body')
-      notification_a = create(:notification, notifiable: proposal_notification_a)
-      notification_b = create(:notification, notifiable: proposal_notification_b)
+      create(:notification, notifiable: proposal_notification_a)
+      create(:notification, notifiable: proposal_notification_b)
 
       visit admin_system_email_view_path('proposal_notification_digest')
 
       expect(page).to have_content('Proposal notifications in')
-      expect(page).to have_link('Proposal A Title', href: notification_url(notification_a))
-      expect(page).to have_link('Proposal B Title', href: notification_url(notification_b))
+      expect(page).to have_link('Proposal A Title', href: proposal_url(proposal_a,
+                                                    anchor: 'tab-notifications'))
+      expect(page).to have_link('Proposal B Title', href: proposal_url(proposal_b,
+                                                    anchor: 'tab-notifications'))
       expect(page).to have_content('Proposal A Notification Body')
       expect(page).to have_content('Proposal B Notification Body')
     end
diff --git a/spec/features/emails_spec.rb b/spec/features/emails_spec.rb
index 4d009bfbc..91f8b31b6 100644
--- a/spec/features/emails_spec.rb
+++ b/spec/features/emails_spec.rb
@@ -306,14 +306,14 @@ feature 'Emails' do
       expect(email).to have_body_text(notification1.notifiable.body)
       expect(email).to have_body_text(proposal1.author.name)
 
-      expect(email).to have_body_text(/#{notification_path(notification1)}/)
+      expect(email).to have_body_text(/#{proposal_path(proposal1, anchor: 'tab-notifications')}/)
       expect(email).to have_body_text(/#{proposal_path(proposal1, anchor: 'comments')}/)
       expect(email).to have_body_text(/#{proposal_path(proposal1, anchor: 'social-share')}/)
 
       expect(email).to have_body_text(proposal2.title)
       expect(email).to have_body_text(notification2.notifiable.title)
       expect(email).to have_body_text(notification2.notifiable.body)
-      expect(email).to have_body_text(/#{notification_path(notification2)}/)
+      expect(email).to have_body_text(/#{proposal_path(proposal2, anchor: 'tab-notifications')}/)
       expect(email).to have_body_text(/#{proposal_path(proposal2, anchor: 'comments')}/)
       expect(email).to have_body_text(/#{proposal_path(proposal2, anchor: 'social-share')}/)
       expect(email).to have_body_text(proposal2.author.name)