Add sqlserver support (#451)

By specifying the type for the params column, we can make sure ActiveRecord knows how to handle this for more databases like sqlserver.

Fixes #450

Author: excid3

CHANGELOG.md

@@ -1,5 +1,8 @@
 ### Unreleased
 
+* Set `:json` type on `:params` column with default to better integrate with ActiveRecord.
+  This fixes sqlserver (and probably other databases). #451
+
 ### 2.3.1
 
 * Skip `ApplicationNotifier` in generator if it already exists

app/models/concerns/noticed/deliverable.rb

@@ -6,17 +6,6 @@ module Deliverable
       class_attribute :bulk_delivery_methods, instance_writer: false, default: {}
       class_attribute :delivery_methods, instance_writer: false, default: {}
       class_attribute :required_param_names, instance_writer: false, default: []
-
-      attribute :params, default: {}
-
-      # Ephemeral notifiers cannot serialize params since they aren't ActiveRecord backed
-      if respond_to? :serialize
-        if Rails.gem_version >= Gem::Version.new("7.1.0.alpha")
-          serialize :params, coder: Coder
-        else
-          serialize :params, Coder
-        end
-      end
     end
 
     class_methods do

app/models/noticed/event.rb

@@ -11,5 +11,16 @@ class Event < ApplicationRecord
     accepts_nested_attributes_for :notifications
 
     scope :newest_first, -> { order(created_at: :desc) }
+
+    attribute :params, :json, default: {}
+
+    # Ephemeral notifiers cannot serialize params since they aren't ActiveRecord backed
+    if respond_to? :serialize
+      if Rails.gem_version >= Gem::Version.new("7.1.0.alpha")
+        serialize :params, coder: Coder
+      else
+        serialize :params, Coder
+      end
+    end
   end
 end