think_feel_do_engine 3.21.2 → 3.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3ff553dad57d0ae0a68326a83dd251c9f44037c2
-  data.tar.gz: ef4e2750dd4cfe73e15445ac1c3303a7fedbf3d8
+  metadata.gz: 4f7415b8925aa5d127cbc80d056d04a6d44ef499
+  data.tar.gz: d892773131b44f665850c68441d6a07cd44bc3d8
 SHA512:
-  metadata.gz: ece5a0cd2dac5b4f0f8ad8f58cdef342323fab606075c5ecfe077d12f9bc7c5e0141f5f763d9704ff15c81f7f36c2792930f81129ee4d0bbda587d3fe227f93b
-  data.tar.gz: 73a185e725ad537b6543f8aae77ccc55b03a78778c725089f0b326cd6d344f72b86339053d31d34a02e15cc0003f10de494012994abc566160b428361c9887cd
+  metadata.gz: 1d5332d7126f454e962345cc75f8d9dc7a64605e1222b60065b75fccba4061889a7f48fbddfc5a91e9b3a32f260d00c74e39c27987bc8111173322053c2f5ea4
+  data.tar.gz: b6194ac2ddb5d08e59bc2b090b40bbe09ab40ab1d5f2d34f59f852761d8a1bb6804e0982ae5797d6a67dcf3a93fb35a36fda1c56d1931bdbccfb140cc8e83cc5

data/app/models/message_sms_notification.rb

@@ -3,6 +3,8 @@
 class MessageSmsNotification
   attr_reader :body, :phone_number
 
+  BLACKLIST_MESSAGE = "The message From/To pair violates a blacklist rule."
+
   def initialize(body:, phone_number:)
     @body = body
     @phone_number = phone_number
@@ -14,10 +16,23 @@ class MessageSmsNotification
     else
       message
     end
+  rescue Twilio::REST::RequestError => e
+    if e.message == BLACKLIST_MESSAGE
+      record_delivery_error!(e.message)
+    else
+      capture_message(e.message)
+    end
   end
 
   private
 
+  def capture_message(message)
+    Raven
+      .capture_message message, extra: {
+        phone_number: phone_number
+      } if defined?(Raven)
+  end
+
   def config
     Rails.application.config
   end
@@ -46,6 +61,15 @@ class MessageSmsNotification
     params
   end
 
+  def record_delivery_error!(message)
+    ThinkFeelDoEngine::SmsDeliveryError
+      .create!(
+        body: body,
+        error_message: message,
+        phone_number: phone_number
+      )
+  end
+
   def sms_client
     @sms_client ||= Twilio::REST::Client.new(
       config.twilio_account_sid,

data/app/models/think_feel_do_engine/reports/sms_delivery_error_record.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+module ThinkFeelDoEngine
+  module Reports
+    # Collects relevant/unsubscribed SMS errors
+    class SmsDeliveryErrorRecord
+      def self.columns
+        %w( body error_message phone_number created_at )
+      end
+
+      def self.all
+        SmsDeliveryError
+          .all
+          .map do |sms_delivery_error|
+            {
+              body: sms_delivery_error.body,
+              error_message: sms_delivery_error.error_message,
+              phone_number: sms_delivery_error.phone_number,
+              created_at: sms_delivery_error.created_at.iso8601
+            }
+          end
+          .flatten
+      end
+    end
+  end
+end

data/app/models/think_feel_do_engine/sms_delivery_error.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+module ThinkFeelDoEngine
+  # Records sms errors when they aren't delivered
+  class SmsDeliveryError < ActiveRecord::Base
+    validates :body,
+              :error_message,
+              :phone_number,
+              presence: true
+  end
+end

data/db/migrate/20160712153753_create_think_feel_do_engine_sms_delivery_errors.rb

@@ -0,0 +1,11 @@
+class CreateThinkFeelDoEngineSmsDeliveryErrors < ActiveRecord::Migration
+  def change
+    create_table :think_feel_do_engine_sms_delivery_errors do |t|
+      t.string :phone_number, null: false
+      t.text :body, null: false
+      t.text :error_message, null: false
+
+      t.timestamps null: false
+    end
+  end
+end

data/lib/think_feel_do_engine/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 # nodoc
 module ThinkFeelDoEngine
-  VERSION = "3.21.2"
+  VERSION = "3.22.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: think_feel_do_engine
 version: !ruby/object:Gem::Version
-  version: 3.21.2
+  version: 3.22.0
 platform: ruby
 authors:
 - Eric Carty-Fickes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-25 00:00:00.000000000 Z
+date: 2016-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -561,12 +561,14 @@ files:
 - app/models/think_feel_do_engine/reports/phq_assessment.rb
 - app/models/think_feel_do_engine/reports/reporter.rb
 - app/models/think_feel_do_engine/reports/site_session.rb
+- app/models/think_feel_do_engine/reports/sms_delivery_error_record.rb
 - app/models/think_feel_do_engine/reports/task_completion.rb
 - app/models/think_feel_do_engine/reports/tool_access.rb
 - app/models/think_feel_do_engine/reports/tool_module.rb
 - app/models/think_feel_do_engine/reports/user_agent.rb
 - app/models/think_feel_do_engine/reports/video_session.rb
 - app/models/think_feel_do_engine/reports/wai_assessment.rb
+- app/models/think_feel_do_engine/sms_delivery_error.rb
 - app/models/thought.rb
 - app/models/thought_pattern.rb
 - app/models/tool_nav_item.rb
@@ -965,6 +967,7 @@ files:
 - db/migrate/20160516143322_remove_experience.rb
 - db/migrate/20160616162347_convert_slide_options.rb
 - db/migrate/20160616201120_convert_event_payloads.rb
+- db/migrate/20160712153753_create_think_feel_do_engine_sms_delivery_errors.rb
 - lib/tasks/lesson_notifications.rake
 - lib/tasks/seed.rake
 - lib/tasks/think_feel_do_engine_tasks.rake