Move NotificationChannel to lib so it isn't eager loaded in apps that…

… don't include ActionCable Fixes #392
excid3 · Jan 30, 2024 · 713598d · 713598d
1 parent 11c1ce5
commit 713598d
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ### 2.1.1
 
 * Fix duplicate column in migrations.
+* Add compatibility for apps that don't use ActionCable.
 
 ### 2.1.0
 

diff --git a/app/views/.keep b/app/views/.keep
diff --git a/lib/noticed.rb b/lib/noticed.rb
@@ -15,6 +15,7 @@ def self.deprecator # :nodoc:
   autoload :Coder, "noticed/coder"
   autoload :DeliveryMethod, "noticed/delivery_method"
   autoload :HasNotifications, "noticed/has_notifications"
+  autoload :NotificationChannel, "noticed/notification_channel"
   autoload :RequiredOptions, "noticed/required_options"
   autoload :Translation, "noticed/translation"
 

diff --git a/app/channels/noticed/notification_channel.rb → lib/noticed/notification_channel.rb b/app/channels/noticed/notification_channel.rb → lib/noticed/notification_channel.rb
@@ -1,5 +1,5 @@
 module Noticed
-  class NotificationChannel < ::ApplicationCable::Channel
+  class NotificationChannel < ApplicationCable::Channel
     def subscribed
       stream_for current_user
     end