notifykit 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d96baaa9ac5a0a64a5f914b4a31daa06a06e2fcc
-  data.tar.gz: 39ac9eda71d9fe5bcfebd394d792f8d5b125b30a
+  metadata.gz: 94dc4739c05ee8b8d65fdaa9a3e0bcbf210e87ba
+  data.tar.gz: 57b5cf46b2de9009b890ce281019533441f791e1
 SHA512:
-  metadata.gz: f3156b96771dbd2806f606571727d8ad0ae82680f3c21fc9019ca478864c4ec975b7d93cfd7db6f43ea6b5b82d711c4bd25d86143090ce5e9f73ba2172626f7b
-  data.tar.gz: 151d51d7e9c410424d61f8f80b51ec646ebaff300b6a250854e297a14e95fb6519ea7e37e60f5e43fb12a15d50104d465c70540a2bf7874c1671c4635e5ce5fa
+  metadata.gz: 017745b6af35344c4ecdc614e19797fb52cfc4b82831488918eab86ead51440fcfd09639a711d67af7ee91b00cce155d3b2db1f5dc48094beaa3ac7876ffbe24
+  data.tar.gz: ac4ab1b66479d0a3f79b097054e5034a2c77b07e41a9e26606d10bbc187bde9c2fa1095c28b45a65826bf81aa2668dee2897a4b53855231656b282007c649926

data/README.md

@@ -37,7 +37,7 @@ emailed it is updated with the rendered text and HTML. Any interaction with
 the notification will create additional database operations.
 
 Even with a modest amount of emails (1000/day) the database would grow
-by 10M.day. This can easily overwhelm a small system.
+by 10Mb.day. This can easily overwhelm a small system.
 
 It is possible to change how rendered email messages are stored. For
 instance you could change to a strategy where only the needed data

data/lib/generators/notifykit/install_generator.rb

@@ -62,8 +62,8 @@ module Notifykit
         route "get    '/terms', to: 'terms#index', as: :terms"
       end
 
-      # Adjust the user
-      inject_into_class "app/models/user.rb", User, "has_many :notifications\n"
+      # Adjust the user, unless it doesn't exist
+      inject_into_class "app/models/user.rb", User, "has_many :notifications\n" rescue nil
 
       # Technically, we aren't inserting this at the end of the class, but the end of the RSpec::Configure
       insert_at_end_of_class "spec/spec_helper.rb", "spec/spec_helper.rb"
@@ -86,14 +86,14 @@ module Notifykit
       source = File.expand_path(find_in_source_paths(source.to_s))
       context = instance_eval('binding')
       content = ERB.new(::File.binread(source), nil, '-', '@output_buffer').result(context)
-      insert_into_file filename, "#{content}\n", before: /\z/
+      insert_into_file filename, "#{content}\n", before: /\z/ rescue nil
     end
 
     def insert_at_end_of_class(filename, source)
       source = File.expand_path(find_in_source_paths(source.to_s))
       context = instance_eval('binding')
       content = ERB.new(::File.binread(source), nil, '-', '@output_buffer').result(context)
-      insert_into_file filename, "#{content}\n", before: /end\n*\z/
+      insert_into_file filename, "#{content}\n", before: /end\n*\z/ rescue nil
     end
 
     def generate_migration(filename)

data/lib/generators/notifykit/templates/app/models/notification.rb

@@ -60,7 +60,7 @@ class Notification < ActiveRecord::Base
   def set_email
     return if !self.deliver_via_email?
 
-    self.email ||= self.user.email rescue nil
+    self.email ||= self.user.try(:email) rescue nil
   end
 
   # The default size is 16 which is 1/64^16, this is protected by

data/lib/generators/notifykit/templates/app/views/notifications_mailer/notify.html.erb

@@ -48,9 +48,17 @@
       padding:0;
       line-height:26px;
     }
+    /* For targeting in Gmail */
+    div[title='content'] {
+      padding: 0 40px 0 40px;
+    }
     div.content {
       padding: 0 40px 0 40px;
     }
+    /* For targeting in Gmail */
+    div[title='content'] p {
+      font-size: 16px;
+    }
     div.content p {
       font-size: 16px;
     }
@@ -106,6 +114,10 @@
       zoom: 1;
     }
 
+    *, *:before, *:after {
+      -moz-box-sizing: border-box; -webkit-box-sizing: border-box; box-sizing: border-box;
+    }
+
     @media only screen and (max-width: 480px) {
       body, #body_style {
         font-size: 24px !important;
@@ -114,10 +126,10 @@
         font-stretch:normal;
         line-height:32px;
       }
-      table[class=table] {width:480px !important;}
+      table[class=table] {width:100% !important;}
 
       /* footer */
-      table[class=footer_1] {width: 405px !important;}
+      table[class=footer_1] {width: 80% !important;}
       td[class=footer_1] {font-size: 16px !important;line-height: 22px !important;}
       td[class=footer_2] {font-size: 16px !important;line-height: 22px !important;}
       td[class=unsubscribe] {font-size: 14px !important;}
@@ -126,7 +138,7 @@
 </head>
 <body style="background:#F7F7F7; color:#7F7F7F; font-size:14px" alink="#000000" link="#000000" bgcolor="#F7F7F7" text="#7F7F7F">
   <div id="body_style">
-    <table width="100%" cellspacing="0" cellpadding="0" border="0" align="center" bgcolor="#F7F7F7">
+    <table width="100%" cellspacing="0" cellpadding="0" border="0" align="center" bgcolor="#dfdfdf">
       <tr>
         <td style="padding:0px 20px 20px 20px;">
           <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
@@ -134,13 +146,15 @@
               <td align="center" style="border-bottom: 1px solid #FFFFFF;">
                 <table width="100%" cellspacing="0" cellpadding="0" border="0">
                   <tr>
-                    <td align="center" class="contains_graphics" style="border-bottom:1px solid #CCCCCC;font-size: 11px;color:#999999;text-align:center;padding:10px 0px;" valign="middle">
+                    <td align="center" class="contains_graphics" style="border-bottom:1px solid #CCCCCC;font-size: 10px;color:#999999;text-align:center;padding:10px 0px;" valign="middle">
                       <span>This email contains graphics. If you don’t see them,</span> <a href="<%= notification_view_url(notification) %>">click here</a>.
                     </td>
                   </tr>
                 </table>
               </td>
             </tr>
+          </table>
+          <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
             <tr>
               <td>
                 <table width="100%" cellspacing="0" cellpadding="0" border="0">
@@ -149,9 +163,11 @@
                       <table cellspacing="0" cellpadding="0" border="0" align="center">
                         <tr>
                           <td align="center">
-                            <a href="<%= append_tracking(root_url) %>">
-                              <%= notification_company_logo %>
-                            </a>
+                            <div style="padding: 16px">
+                              <a href="<%= append_tracking(root_url) %>">
+                                <%= notification_company_logo %>
+                              </a>
+                            </div>
                           </td>
                         </tr>
                       </table>
@@ -160,21 +176,27 @@
                 </table>
               </td>
             </tr>
+          </table>
+          <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
             <tr>
               <td class="bar" width="100%" height="0" style="border-bottom:4px solid #666666;">
               </td>
             </tr>
+          </table>
+          <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
             <tr>
               <td>
                 <table width="100%" cellspacing="0" cellpadding="0" border="0" bgcolor="#FFFFFF">
                   <tr>
-                    <td style="border: 1px solid #CCCCCC; border-top: none; border-bottom: none; padding: 40px 0px;" class="wrapper_td">
+                    <td style="border: 1px solid #CCCCCC; border-top: none; border-bottom: none; padding: 40px 40px;" class="wrapper_td">
                       <%= render partial: notification.kind %>
                     </td>
                   </tr>
                 </table>
               </td>
             </tr>
+          </table>
+          <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
             <tr>
               <td style="border:1px solid #CCCCCC;">
                 <table width="100%" cellspacing="0" cellpadding="0" border="0" bgcolor="#FFFFFF" align="center">
@@ -193,6 +215,8 @@
                 </table>
               </td>
             </tr>
+          </table>
+          <table class="table" width="600" cellspacing="0" cellpadding="0" border="0" align="center">
             <tr>
               <td align="center">
                 <table class="footer2" width="100%" cellspacing="0" cellpadding="0" border="0" align="center">
@@ -202,6 +226,8 @@
                       <%= notification_company_address %>
                     </td>
                   </tr>
+               </table>
+                <table class="footer2" width="100%" cellspacing="0" cellpadding="0" border="0" align="center">
                   <tr>
                     <td class="unsubscribe" align="center" style="text-align:center;font-size:11px;color:#999999">
                       <% unless notification.transactional? -%><a href="<%= notification_unsubscribe_url(notification) %>">Unsubscribe</a><br><% end -%>
@@ -218,4 +244,4 @@
     </table>
   </div>
 </body>
-</html>
+

data/lib/notifykit/version.rb

@@ -1,3 +1,3 @@
 module Notifykit
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,69 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: notifykit
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Jeff Rafter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-29 00:00:00.000000000 Z
+date: 2014-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: factory_girl_rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: A notification system for your Rails app
@@ -73,7 +73,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - CHANGELOG.md
 - FEATURES.md
 - Gemfile
@@ -113,17 +113,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Notifykit is a generator that will install a notifications model, mailer,