v0.2.3 (#12)

* Update webpack dev server

* Update email copy

* Add pending playbacks path

Author: suninthesky

app/assets/stylesheets/minireset.sass

@@ -80,3 +80,4 @@ td,
 th
   padding: 0
   text-align: left
+  border-color: #ccc !important

app/controllers/playbacks_controller.rb

@@ -1,7 +1,11 @@
 class PlaybacksController < ApplicationController
-  before_action :authenticate_user!, except: %[show]
+  before_action :authenticate_user!, except: %i[show pending]
   before_action :set_playback, only: %i[show edit update destroy]
 
+  def pending
+    @playbacks = Playback.where(comments_count: params[:comments] || 0)
+  end
+
   def index
     @playbacks = Playback.all
   end

app/mailers/application_mailer.rb

@@ -1,4 +1,4 @@
 class ApplicationMailer < ActionMailer::Base
-  default from: 'no-reply@fusebox-playbacks.herokuapp.com'
+  default from: 'CAST <designhops@wearecast.org.uk>'
   layout 'mailer'
 end

app/models/comment.rb

@@ -4,10 +4,15 @@ class Comment < ApplicationRecord
   validates :author, :body, presence: true
 
   before_validation :mark_step_as_done!
+  after_create :update_comments_count!
 
   private
 
     def mark_step_as_done!
       step.update_column(:completed_on, Date.today) if done
     end
+
+    def update_comments_count!
+      Playback.increment_counter(:comments_count, commentable.playback_id)
+    end
 end

app/views/comments_mailer/new_comment.html.erb

@@ -1,11 +1,13 @@
 <h1><%= @comment.author %> <%= comment_done_message(@comment) %></h1>
 
-<h3 style="margin: 0;">Step</h3>
+<h3 style="margin: 0;">Section</h3>
 <p style="margin: 5px 0 20px;"><%= raw @comment.commentable.name %></p>
 
 <h3 style="margin: 0 0 5px;">Comment</h3>
 <div style="background: #f2f2f2; padding: 20px; margin-bottom: 20px;">
   <%= raw @comment.body %>
 </div>
 
-<h3><%= link_to 'Reply here', new_comment_url(@comment.commentable) %></h3>
+<h3><%= link_to 'Reply here', new_comment_url(@comment.commentable) %></h3>
+
+<small>This is an automated email for your records.</small>

app/views/playbacks/pending.html.erb

@@ -0,0 +1,23 @@
+<div class="maxw1050 mx-auto px15">
+  <h1 class="h2 bold my40">Pending Playbacks</h1>
+
+  <table>
+    <thead>
+      <tr>
+        <th>Organisation Name</th>
+        <th>No. of comments</th>
+        <th></th>
+      </tr>
+    </thead>
+
+    <tbody>
+      <% @playbacks.each do |playback| %>
+        <tr>
+          <td><%= playback.organisation_name %> <small>by <%= playback.author_name %></small></td>
+          <td><%= playback.comments_count %></td>
+          <td><%= link_to 'View', playback %></td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+</div>

config/routes.rb

@@ -3,6 +3,8 @@
 
   resources :playbacks
 
+  get '/pending/playbacks/(:count)', to: 'playbacks#pending', as: 'pending_playbacks'
+
   get '/support/:commentable_id/comments', to: 'comments#new', as: 'new_comment'
   post '/support/:commentable_id/comments', to: 'comments#create'
 

db/migrate/20190206170022_add_counter_cache_to_playbacks.rb

@@ -0,0 +1,14 @@
+class AddCounterCacheToPlaybacks < ActiveRecord::Migration[5.2]
+  def change
+    add_column :playbacks, :comments_count, :integer, null: false, default: 0
+
+    reversible do |dir|
+      dir.up do
+        Section.joins(:comments).group(:playback_id).count(:commentable_id).each do |id, count|
+          Playback.find(id).update(comments_count: count)
+          print '.'
+        end
+      end
+    end
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_11_27_160118) do
+ActiveRecord::Schema.define(version: 2019_02_06_170022) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -52,6 +52,8 @@
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.string "source"
+    t.string "email"
+    t.integer "comments_count", default: 0, null: false
   end
 
   create_table "sections", force: :cascade do |t|

package.json

@@ -8,6 +8,6 @@
     "vue-template-compiler": "^2.5.17"
   },
   "devDependencies": {
-    "webpack-dev-server": "2.11.2"
+    "webpack-dev-server": ">=3.1.11"
   }
 }