Subtype fixes

app/notifications/base_notification.rb

@@ -6,7 +6,7 @@ class BaseNotification < Noticed::Base
 
   def self.send_notifications(params, later: false)
     recipients_class =
-      "Recipients::#{NotificationExtends::NOTIFICATIONS_TYPES[params[:type]][:recipients_module]}".constantize
+      "Recipients::#{NotificationExtends::NOTIFICATIONS_TYPES[subtype || params[:type]][:recipients_module]}".constantize
     recipients_class.new(params).recipients.each do |recipient|
       if later
         with(params).deliver_later(recipient)
@@ -16,8 +16,10 @@ class BaseNotification < Noticed::Base
     end
   end
 
+  def self.subtype; end
+
   def subtype
-    params[:type]
+    self.class.subtype || params[:type]
   end
 
   def subject; end

app/notifications/general_notification.rb

@@ -1,40 +1,15 @@
 # frozen_string_literal: true
 
-# To deliver this notification:
-#
-# GeneralNotification.with(post: @post).deliver_later(current_user)
-# GeneralNotification.with(post: @post).deliver(current_user)
-
 class GeneralNotification < BaseNotification
-  # Add your delivery methods
-  #
-  # deliver_by :email, mailer: "UserMailer"
-  # deliver_by :slack
-  # deliver_by :custom, class: "MyDeliveryMethod"
-
-  # Add required params
-  #
-  # param :post
-
-  # Define helper methods to make rendering easier.
-  #
   def message
-    # if params[:legacy]
     params[:message]
-    # else
-    # new logic
-    # end
   end
 
   def title
-    # if params[:legacy]
     params[:title]
-    # else
-    # new logic
-    # end
   end
-  #
+
-  # def url
+  def subtype
-  #   post_path(params[:post])
+    params[:type]
-  # end
+  end
 end

app/notifications/low_stock_notification.rb

@@ -8,7 +8,7 @@ class LowStockNotification < BaseNotification
     )
   end
 
-  def subtype
+  def self.subtype
     :item_low_stock_reminder
   end