From abaf80e5c4832dd926552cb0e7de05f7341457e0 Mon Sep 17 00:00:00 2001
From: Hugo Peixoto <hugo.peixoto@gmail.com>
Date: Thu, 6 Oct 2022 15:50:38 +0100
Subject: [PATCH] Add notification index page

---
 app/controllers/notifications_controller.rb |  8 +++-
 app/views/layouts/application.html.erb      |  1 +
 app/views/notifications/index.html.erb      | 53 +++++++++++++++++++++
 config/locales/pt.yml                       | 22 +++++----
 config/routes.rb                            |  2 +-
 5 files changed, 76 insertions(+), 10 deletions(-)
 create mode 100644 app/views/notifications/index.html.erb

diff --git a/app/controllers/notifications_controller.rb b/app/controllers/notifications_controller.rb
index bfbe148..c3a16f8 100644
--- a/app/controllers/notifications_controller.rb
+++ b/app/controllers/notifications_controller.rb
@@ -1,6 +1,12 @@
 class NotificationsController < ApplicationController
   before_action :require_login
-  before_action :set_notification
+  before_action :set_notification, only: [:deliver]
+
+  # GET /notifications
+  def index
+    @scheduled = Notification.where(status: 'scheduled').order(to_be_sent_on: :asc)
+    @sent = Notification.where(status: 'sent').order(sent_at: :desc)
+  end
 
   # POST /notifications/1/deliver
   def deliver
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index 5bb562a..4636d6a 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -16,6 +16,7 @@
         <ul>
           <li><%= link_to t('navigation.members'), members_path %></li>
           <li><%= link_to t('navigation.board'), edit_board_path %></li>
+          <li><%= link_to t('navigation.notifications'), notifications_path %></li>
           <li>
             <%= button_to sign_out_path, method: :delete do %>
               Sign out of <%= current_user.email %>
diff --git a/app/views/notifications/index.html.erb b/app/views/notifications/index.html.erb
new file mode 100644
index 0000000..f2cff23
--- /dev/null
+++ b/app/views/notifications/index.html.erb
@@ -0,0 +1,53 @@
+<h1><%= t('notifications.index.title') %></h1>
+
+<details>
+  <summary><h2 style="display: inline"><%= t('notifications.index.scheduled') %></h2></summary>
+  <table class='zebra'>
+    <tr>
+      <th><%= t('notifications.attributes.to_be_sent_on') %></th>
+      <th><%= t('notifications.attributes.template') %></th>
+      <th><%= t('notifications.attributes.member') %></th>
+      <th><%= t('members.show.contribution_actions') %></th>
+    </tr>
+    <% @scheduled.each do |notification| %>
+      <tr>
+        <td><%= notification.to_be_sent_on %></td>
+        <td><code><%= notification.template %></code></td>
+        <td><%= link_to notification.member do %><%= notification.member.display_name %> &lt;<%= notification.member.email %>&gt;<% end %></td>
+        <td>
+          <% if notification.status == 'scheduled' %>
+            <%= form_with url: deliver_notification_path(notification) do |form| %>
+              <%= form.submit t('members.show.actions.deliver_notification') %>
+            <% end %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </table>
+</details>
+
+<details>
+  <summary><h2 style="display: inline"><%= t('notifications.index.sent') %></h2></summary>
+  <table class='zebra'>
+    <tr>
+      <th><%= t('notifications.attributes.to_be_sent_on') %></th>
+      <th><%= t('notifications.attributes.template') %></th>
+      <th><%= t('notifications.attributes.status') %></th>
+      <th><%= t('members.show.contribution_actions') %></th>
+    </tr>
+    <% @sent.each do |notification| %>
+      <tr>
+        <td><%= notification.to_be_sent_on %></td>
+        <td><code><%= notification.template %></code></td>
+        <td><%= notification_status(notification.status) %></td>
+        <td>
+          <% if notification.status == 'scheduled' %>
+            <%= form_with url: deliver_notification_path(notification) do |form| %>
+              <%= form.submit t('members.show.actions.deliver_notification') %>
+            <% end %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </table>
+</details>
diff --git a/config/locales/pt.yml b/config/locales/pt.yml
index c900b30..7fe654c 100644
--- a/config/locales/pt.yml
+++ b/config/locales/pt.yml
@@ -4,6 +4,8 @@ pt:
     negative: "Não"
   navigation:
     members: "Lista de membros"
+    board: "Direcção"
+    notifications: "Notificações"
   members:
     delete:
       confirmation_message: |
@@ -68,14 +70,6 @@ pt:
       unemployed: "Desempregado"
       student: "Estudante"
       retired: "Reformado"
-  notifications:
-    attributes:
-      to_be_sent_on: Data de envio
-      template: Modelo
-      status: Estado
-    status:
-      sent: Enviada
-      scheduled: Agendada
   contributions:
     attributes:
       amount: Montante
@@ -121,3 +115,15 @@ pt:
       warning: |
         Atenção: qualquer endereço que ficar omisso verá a sua conta suspensa e
         deixará de poder aceder a este sistema.
+  notifications:
+    index:
+      title: "Notificações"
+      scheduled: "Agendadas"
+      sent: "Enviadas"
+    attributes:
+      to_be_sent_on: Data de envio
+      template: Modelo
+      status: Estado
+    status:
+      sent: Enviada
+      scheduled: Agendada
diff --git a/config/routes.rb b/config/routes.rb
index 95b06d8..96882fd 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -19,7 +19,7 @@ Rails.application.routes.draw do
     end
   end
 
-  resources :notifications, only: [] do
+  resources :notifications, only: [:index] do
     member do
       post :deliver
     end