activity_notification 1.5.1 → 1.6.0

data/lib/activity_notification/apis/subscription_api.rb

@@ -188,14 +188,14 @@ module ActivityNotification
 
     protected
 
-      # Validates subscribing_to_email cannot be true when subscribing isfalse.
+      # Validates subscribing_to_email cannot be true when subscribing is false.
       def subscribing_to_email_cannot_be_true_when_subscribing_is_false
         if !subscribing && subscribing_to_email?
           errors.add(:subscribing_to_email, "cannot be true when subscribing is false")
         end
       end
 
-      # Validates subscribing_to_optional_target cannot be true when subscribing isfalse.
+      # Validates subscribing_to_optional_target cannot be true when subscribing is false.
       def subscribing_to_optional_target_cannot_be_true_when_subscribing_is_false
         optional_target_names.each do |optional_target_name|
           if !subscribing && subscribing_to_optional_target?(optional_target_name)

data/lib/activity_notification/controllers/devise_authentication_controller.rb

@@ -27,6 +27,22 @@ module ActivityNotification
         end
       end
 
+      # Sets @target instance variable from request parameters.
+      # This method override super (ActivityNotiication::CommonController#set_target)
+      # to set devise authenticated target when the target_id params is not specified.
+      # @api protected
+      # @return [Object] Target instance (Returns HTTP 400 when request parameters are not enough)
+      def set_target
+        target_type = params[:target_type]
+        if params[:target_id].blank? && params["#{target_type.to_resource_name}_id"].blank?
+          target_class = target_type.to_model_class
+          current_resource_method_name = "current_#{params[:devise_type].to_resource_name}"
+          params[:target_id] = target_class.resolve_current_devise_target(send(current_resource_method_name))
+          render(plain: "403 Forbidden: Unauthenticated as default target", status: 403) and return if params[:target_id].blank?
+        end
+        super
+      end
+
       # Authenticate the target of requested notification with authenticated devise resource.
       # @api protected
       # @todo Needs to call authenticate method by more secure way

data/lib/activity_notification/helpers/view_helpers.rb

@@ -40,6 +40,8 @@ module ActivityNotification
     # @option options [String]         :notification_layout    (nil)       Layout template name of the notification index content
     # @option options [String]         :fallback               (nil)       Fallback template to use when MissingTemplate is raised. Set :text to use i18n text as fallback.
     # @option options [String]         :partial                ('index')   Partial template name of the partial index
+    # @option options [String]         :routing_scope          (nil)       Routing scope for notification and subscription routes
+    # @option options [Boolean]        :devise_default_routes  (false)     If links in default views will be handles as devise default routes
     # @option options [String]         :layout                 (nil)       Layout template name of the partial index
     # @option options [Integer]        :limit                  (nil)       Limit to query for notifications
     # @option options [Boolean]        :reverse                (false)     If notification index will be ordered as earliest first
@@ -51,7 +53,7 @@ module ActivityNotification
     # @option options [String]         :filtered_by_key        (nil)       Key of the notification for filter
     # @option options [Array]          :custom_filter          (nil)       Custom notification filter (e.g. ["created_at >= ?", time.hour.ago])
     # @return [String] Rendered view or text as string
-    def render_notification_of target, options = {}
+    def render_notification_of(target, options = {})
       return unless target.is_a? ActivityNotification::Target
 
       # Prepare content for notifications index
@@ -77,7 +79,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def notifications_path_for(target, params = {})
-      send("#{target.to_resource_name}_notifications_path", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}notifications_path", options) :
+        send("#{routing_scope(options)}#{target.to_resource_name}_notifications_path", target, options)
     end
 
     # Returns notification_path for the notification
@@ -88,7 +93,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def notification_path_for(notification, params = {})
-      send("#{notification.target.to_resource_name}_notification_path", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}notification_path", notification, options) :
+        send("#{routing_scope(options)}#{notification.target.to_resource_name}_notification_path", notification.target, notification, options)
     end
 
     # Returns move_notification_path for the target of specified notification
@@ -99,7 +107,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def move_notification_path_for(notification, params = {})
-      send("move_#{notification.target.to_resource_name}_notification_path", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("move_#{routing_scope(options)}notification_path", notification, options) :
+        send("move_#{routing_scope(options)}#{notification.target.to_resource_name}_notification_path", notification.target, notification, options)
     end
 
     # Returns open_notification_path for the target of specified notification
@@ -110,7 +121,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def open_notification_path_for(notification, params = {})
-      send("open_#{notification.target.to_resource_name}_notification_path", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("open_#{routing_scope(options)}notification_path", notification, options) :
+        send("open_#{routing_scope(options)}#{notification.target.to_resource_name}_notification_path", notification.target, notification, options)
     end
 
     # Returns open_all_notifications_path for the target
@@ -121,7 +135,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def open_all_notifications_path_for(target, params = {})
-      send("open_all_#{target.to_resource_name}_notifications_path", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("open_all_#{routing_scope(options)}notifications_path", options) :
+        send("open_all_#{routing_scope(options)}#{target.to_resource_name}_notifications_path", target, options)
     end
 
     # Returns notifications_url for the target
@@ -132,7 +149,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def notifications_url_for(target, params = {})
-      send("#{target.to_resource_name}_notifications_url", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}notifications_url", options) :
+        send("#{routing_scope(options)}#{target.to_resource_name}_notifications_url", target, options)
     end
 
     # Returns notification_url for the target of specified notification
@@ -143,7 +163,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def notification_url_for(notification, params = {})
-      send("#{notification.target.to_resource_name}_notification_url", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}notification_url", notification, options) :
+        send("#{routing_scope(options)}#{notification.target.to_resource_name}_notification_url", notification.target, notification, options)
     end
 
     # Returns move_notification_url for the target of specified notification
@@ -154,7 +177,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def move_notification_url_for(notification, params = {})
-      send("move_#{notification.target.to_resource_name}_notification_url", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("move_#{routing_scope(options)}notification_url", notification, options) :
+        send("move_#{routing_scope(options)}#{notification.target.to_resource_name}_notification_url", notification.target, notification, options)
     end
 
     # Returns open_notification_url for the target of specified notification
@@ -165,7 +191,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def open_notification_url_for(notification, params = {})
-      send("open_#{notification.target.to_resource_name}_notification_url", notification.target, notification, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("open_#{routing_scope(options)}notification_url", notification, options) :
+        send("open_#{routing_scope(options)}#{notification.target.to_resource_name}_notification_url", notification.target, notification, options)
     end
 
     # Returns open_all_notifications_url for the target of specified notification
@@ -176,7 +205,10 @@ module ActivityNotification
     # @todo Needs any other better implementation
     # @todo Must handle devise namespace
     def open_all_notifications_url_for(target, params = {})
-      send("open_all_#{target.to_resource_name}_notifications_url", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("open_all_#{routing_scope(options)}notifications_url", options) :
+        send("open_all_#{routing_scope(options)}#{target.to_resource_name}_notifications_url", target, options)
     end
 
     # Returns subscriptions_path for the target
@@ -186,7 +218,10 @@ module ActivityNotification
     # @return [String] subscriptions_path for the target
     # @todo Needs any other better implementation
     def subscriptions_path_for(target, params = {})
-      send("#{target.to_resource_name}_subscriptions_path", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}subscriptions_path", options) :
+        send("#{routing_scope(options)}#{target.to_resource_name}_subscriptions_path", target, options)
     end
 
     # Returns subscription_path for the subscription
@@ -196,7 +231,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def subscription_path_for(subscription, params = {})
-      send("#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}subscription_path", subscription, options) :
+        send("#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
 
     # Returns subscribe_subscription_path for the target of specified subscription
@@ -206,7 +244,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def subscribe_subscription_path_for(subscription, params = {})
-      send("subscribe_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("subscribe_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :subscribe_path_for, :subscribe_subscription_path_for
 
@@ -217,7 +258,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_subscription_path_for(subscription, params = {})
-      send("unsubscribe_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("unsubscribe_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_path_for, :unsubscribe_subscription_path_for
 
@@ -228,7 +272,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def subscribe_to_email_subscription_path_for(subscription, params = {})
-      send("subscribe_to_email_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_to_email_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("subscribe_to_email_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :subscribe_to_email_path_for, :subscribe_to_email_subscription_path_for
 
@@ -239,7 +286,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_to_email_subscription_path_for(subscription, params = {})
-      send("unsubscribe_to_email_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_to_email_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("unsubscribe_to_email_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_to_email_path_for, :unsubscribe_to_email_subscription_path_for
 
@@ -250,7 +300,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def subscribe_to_optional_target_subscription_path_for(subscription, params = {})
-      send("subscribe_to_optional_target_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_to_optional_target_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("subscribe_to_optional_target_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :subscribe_to_optional_target_path_for, :subscribe_to_optional_target_subscription_path_for
 
@@ -261,7 +314,10 @@ module ActivityNotification
     # @return [String] subscription_path for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_to_optional_target_subscription_path_for(subscription, params = {})
-      send("unsubscribe_to_optional_target_#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_to_optional_target_#{routing_scope(options)}subscription_path", subscription, options) :
+        send("unsubscribe_to_optional_target_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_path", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_to_optional_target_path_for, :unsubscribe_to_optional_target_subscription_path_for
 
@@ -272,7 +328,10 @@ module ActivityNotification
     # @return [String] subscriptions_url for the target
     # @todo Needs any other better implementation
     def subscriptions_url_for(target, params = {})
-      send("#{target.to_resource_name}_subscriptions_url", target, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}subscriptions_url", options) :
+        send("#{routing_scope(options)}#{target.to_resource_name}_subscriptions_url", target, options)
     end
 
     # Returns subscription_url for the subscription
@@ -282,7 +341,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def subscription_url_for(subscription, params = {})
-      send("#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("#{routing_scope(options)}subscription_url", subscription, options) :
+        send("#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
 
     # Returns subscribe_subscription_url for the target of specified subscription
@@ -292,7 +354,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def subscribe_subscription_url_for(subscription, params = {})
-      send("subscribe_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("subscribe_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :subscribe_url_for, :subscribe_subscription_url_for
 
@@ -303,7 +368,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_subscription_url_for(subscription, params = {})
-      send("unsubscribe_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("unsubscribe_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_url_for, :unsubscribe_subscription_url_for
 
@@ -314,7 +382,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def subscribe_to_email_subscription_url_for(subscription, params = {})
-      send("subscribe_to_email_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_to_email_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("subscribe_to_email_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :subscribe_to_email_url_for, :subscribe_to_email_subscription_url_for
 
@@ -325,7 +396,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_to_email_subscription_url_for(subscription, params = {})
-      send("unsubscribe_to_email_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_to_email_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("unsubscribe_to_email_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_to_email_url_for, :unsubscribe_to_email_subscription_url_for
 
@@ -336,7 +410,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def subscribe_to_optional_target_subscription_url_for(subscription, params = {})
-      send("subscribe_to_optional_target_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("subscribe_to_optional_target_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("subscribe_to_optional_target_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :subscribe_to_optional_target_url_for, :subscribe_to_optional_target_subscription_url_for
 
@@ -347,7 +424,10 @@ module ActivityNotification
     # @return [String] subscription_url for the subscription
     # @todo Needs any other better implementation
     def unsubscribe_to_optional_target_subscription_url_for(subscription, params = {})
-      send("unsubscribe_to_optional_target_#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, params)
+      options = params.dup
+      options.delete(:devise_default_routes) ?
+        send("unsubscribe_to_optional_target_#{routing_scope(options)}subscription_url", subscription, options) :
+        send("unsubscribe_to_optional_target_#{routing_scope(options)}#{subscription.target.to_resource_name}_subscription_url", subscription.target, subscription, options)
     end
     alias_method :unsubscribe_to_optional_target_url_for, :unsubscribe_to_optional_target_subscription_url_for
 
@@ -441,6 +521,12 @@ module ActivityNotification
         [root, path].map(&:to_s).join('/')
       end
 
+      # Prepare routing scope from options
+      # @api private
+      def routing_scope(options = {})
+        options[:routing_scope] ? options.delete(:routing_scope).to_s + '_' : ''
+      end
+
   end
 
   ActionView::Base.class_eval { include ViewHelpers }

data/lib/activity_notification/models/concerns/subscriber.rb

@@ -59,7 +59,8 @@ module ActivityNotification
       subscription.subscribing_to_email? ?
         subscription.assign_attributes(subscribing_to_email: true, subscribed_to_email_at: created_at) :
         subscription.assign_attributes(subscribing_to_email: false, unsubscribed_to_email_at: created_at)
-      optional_targets = subscription.optional_targets
+      subscription.optional_targets = (subscription.optional_targets || {}).with_indifferent_access
+      optional_targets = {}.with_indifferent_access
       subscription.optional_target_names.each do |optional_target_name|
         optional_targets = subscription.subscribing_to_optional_target?(optional_target_name) ?
           optional_targets.merge(

data/lib/activity_notification/models/concerns/target.rb

@@ -20,6 +20,7 @@ module ActivityNotification
                       :_batch_notification_email_allowed,
                       :_notification_subscription_allowed,
                       :_notification_devise_resource,
+                      :_notification_current_devise_target,
                       :_printable_notification_target_name
       set_target_class_defaults
     end
@@ -39,6 +40,7 @@ module ActivityNotification
         self._batch_notification_email_allowed   = ActivityNotification.config.email_enabled
         self._notification_subscription_allowed  = ActivityNotification.config.subscription_enabled
         self._notification_devise_resource       = ->(model) { model }
+        self._notification_current_devise_target = ->(current_resource) { current_resource }
         self._printable_notification_target_name = :printable_name
         nil
       end
@@ -130,6 +132,15 @@ module ActivityNotification
         }.to_h
       end
 
+      # Resolves current authenticated target by devise authentication from current resource signed in with Devise.
+      # This method is able to be overriden.
+      #
+      # @param [Object] current_resource Current resource signed in with Devise
+      # @return [Object] Current authenticated target by devise authentication
+      def resolve_current_devise_target(current_resource)
+        _notification_current_devise_target.call(current_resource)
+      end
+
       # Returns if subscription management is allowed for this target type.
       # @return [Boolean] If subscription management is allowed for this target type
       def subscription_enabled?

data/lib/activity_notification/rails/routes.rb

@@ -26,6 +26,25 @@ module ActionDispatch::Routing
     #     open_user_notification      POST   /users/:user_id/notifications/:id/open(.:format)
     #       { controller:"activity_notification/notifications", action:"open", target_type:"users" }
     #
+    # You can also configure notification routes with scope like this:
+    #   scope :myscope, as: :myscope do
+    #     notify_to :users, routing_scope: :myscope
+    #   end
+    # This routing_scope option creates the needed routes with specified scope like this:
+    #   # Notification routes
+    #     myscope_user_notifications          GET    /myscope/users/:user_id/notifications(.:format)
+    #       { controller:"activity_notification/notifications", action:"index", target_type:"users", routing_scope: :myscope }
+    #     myscope_user_notification           GET    /myscope/users/:user_id/notifications/:id(.:format)
+    #       { controller:"activity_notification/notifications", action:"show", target_type:"users", routing_scope: :myscope }
+    #     myscope_user_notification           DELETE /myscope/users/:user_id/notifications/:id(.:format)
+    #       { controller:"activity_notification/notifications", action:"destroy", target_type:"users", routing_scope: :myscope }
+    #     open_all_myscope_user_notifications POST   /myscope/users/:user_id/notifications/open_all(.:format)
+    #       { controller:"activity_notification/notifications", action:"open_all", target_type:"users", routing_scope: :myscope }
+    #     move_myscope_user_notification      GET    /myscope/users/:user_id/notifications/:id/move(.:format)
+    #       { controller:"activity_notification/notifications", action:"move", target_type:"users", routing_scope: :myscope }
+    #     open_myscope_user_notification      POST   /myscope/users/:user_id/notifications/:id/open(.:format)
+    #       { controller:"activity_notification/notifications", action:"open", target_type:"users", routing_scope: :myscope }
+    #
     # When you use devise authentication and you want make notification targets assciated with devise,
     # you can create as follows in your routes:
     #   notify_to :users, with_devise: :users
@@ -44,6 +63,23 @@ module ActionDispatch::Routing
     #     open_user_notification      POST   /users/:user_id/notifications/:id/open(.:format)
     #       { controller:"activity_notification/notifications_with_devise", action:"open", target_type:"users", devise_type:"users" }
     #
+    # When you use with_devise option and you want to make simple default routes as follows, you can use devise_default_routes option:
+    #   notify_to :users, with_devise: :users, devise_default_routes: true
+    # These with_devise and devise_default_routes options create the needed routes assciated with authenticated devise resource as the default target
+    #   # Notification with default devise routes
+    #     user_notifications          GET    /notifications(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"index", target_type:"users", devise_type:"users" }
+    #     user_notification           GET    /notifications/:id(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"show", target_type:"users", devise_type:"users" }
+    #     user_notification           DELETE /notifications/:id(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"destroy", target_type:"users", devise_type:"users" }
+    #     open_all_user_notifications POST   /notifications/open_all(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"open_all", target_type:"users", devise_type:"users" }
+    #     move_user_notification      GET    /notifications/:id/move(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"move", target_type:"users", devise_type:"users" }
+    #     open_user_notification      POST   /notifications/:id/open(.:format)
+    #       { controller:"activity_notification/notifications_with_devise", action:"open", target_type:"users", devise_type:"users" }
+    #
     # When you would like to define subscription management paths with notification paths,
     # you can create as follows in your routes:
     #   notify_to :users, with_subscription: true
@@ -60,32 +96,36 @@ module ActionDispatch::Routing
     #   notify_to :users, with_devise: :users
     # @example Define notification paths including subscription paths
     #   notify_to :users, with_subscription: true
+    # @example Integrated with Devise authentication as simple default routes including subscription management
+    #   notify_to :users, with_devise: :users, devise_default_routes: true, with_subscription: true
+    # @example Integrated with Devise authentication as simple default routes with scope including subscription management
+    #   scope :myscope, as: :myscope do
+    #     notify_to :myscope, with_devise: :users, devise_default_routes: true, with_subscription: true, routing_scope: :myscope
+    #   end
     #
     # @overload notify_to(*resources, *options)
     #   @param          [Symbol]       resources Resources to notify
-    #   @option options [Symbol]       :with_devise       (false)          Devise resources name for devise integration. Devise integration will be enabled by this option.
-    #   @option options [Hash|Boolean] :with_subscription (false)          Subscription path options to define subscription management paths with notification paths. Calls subscribed_by routing when truthy value is passed as this option.
-    #   @option options [String]       :model             (:notifications) Model name of notifications
-    #   @option options [String]       :controller        ("activity_notification/notifications" | activity_notification/notifications_with_devise") :controller option as resources routing
-    #   @option options [Symbol]       :as                (nil)            :as option as resources routing
-    #   @option options [Array]        :only              (nil)            :only option as resources routing
-    #   @option options [Array]        :except            (nil)            :except option as resources routing
+    #   @option options [String]       :routing_scope         (nil)            Routing scope for notification routes
+    #   @option options [Symbol]       :with_devise           (false)          Devise resources name for devise integration. Devise integration will be enabled by this option.
+    #   @option options [Boolean]      :devise_default_routes (false)          Whether you will create routes as device default routes assciated with authenticated devise resource as the default target
+    #   @option options [Hash|Boolean] :with_subscription     (false)          Subscription path options to define subscription management paths with notification paths. Calls subscribed_by routing when truthy value is passed as this option.
+    #   @option options [String]       :model                 (:notifications) Model name of notifications
+    #   @option options [String]       :controller            ("activity_notification/notifications" | activity_notification/notifications_with_devise") :controller option as resources routing
+    #   @option options [Symbol]       :as                    (nil)            :as option as resources routing
+    #   @option options [Array]        :only                  (nil)            :only option as resources routing
+    #   @option options [Array]        :except                (nil)            :except option as resources routing
     # @return [ActionDispatch::Routing::Mapper] Routing mapper instance
     def notify_to(*resources)
       options = create_options(:notifications, resources.extract_options!, [:new, :create, :edit, :update])
 
       resources.each do |target|
-        self.resources target, only: :none do
-          options[:defaults] = { target_type: target.to_s }.merge(options[:devise_defaults])
-          resources_options = options.select { |key, _| [:with_devise, :with_subscription, :subscription_option, :model, :devise_defaults].exclude? key }
-          self.resources options[:model], resources_options do
-            collection do
-              post :open_all unless ignore_path?(:open_all, options)
-            end
-            member do
-              get  :move     unless ignore_path?(:move, options)
-              post :open     unless ignore_path?(:open, options)
-            end
+        options[:defaults] = { target_type: target.to_s }.merge(options[:devise_defaults])
+        resources_options = options.select { |key, _| [:with_devise, :devise_default_routes, :with_subscription, :subscription_option, :model, :devise_defaults].exclude? key }
+        if options[:with_devise].present? && options[:devise_default_routes].present?
+          create_notification_routes options, resources_options
+        else
+          self.resources target, only: :none do
+            create_notification_routes options, resources_options
           end
         end
 
@@ -125,9 +165,36 @@ module ActionDispatch::Routing
     #     unsubscribe_to_optional_target_user_subscription  POST   /users/:user_id/subscriptions/:id/unsubscribe_to_optional_target(.:format)
     #       { controller:"activity_notification/subscriptions", action:"unsubscribe_to_optional_target", target_type:"users" }
     #
+    # You can also configure notification routes with scope like this:
+    #   scope :myscope, as: :myscope do
+    #     subscribed_by :users, routing_scope: :myscope
+    #   end
+    # This routing_scope option creates the needed routes with specified scope like this:
+    #   # Subscription routes
+    #     myscope_user_subscriptions                                GET    /myscope/users/:user_id/subscriptions(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"index", target_type:"users", routing_scope: :myscope }
+    #     myscope_user_subscription                                 GET    /myscope/users/:user_id/subscriptions/:id(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"show", target_type:"users", routing_scope: :myscope }
+    #     open_all_myscope_user_subscriptions                       POST   /myscope/users/:user_id/subscriptions(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"create", target_type:"users", routing_scope: :myscope }
+    #     myscope_user_subscription                                 DELETE /myscope/users/:user_id/subscriptions/:id(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"destroy", target_type:"users", routing_scope: :myscope }
+    #     subscribe_myscope_user_subscription                       POST   /myscope/users/:user_id/subscriptions/:id/subscribe(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"subscribe", target_type:"users", routing_scope: :myscope }
+    #     unsubscribe_myscope_user_subscription                     POST   /myscope/users/:user_id/subscriptions/:id/unsubscribe(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"unsubscribe", target_type:"users", routing_scope: :myscope }
+    #     subscribe_to_email_myscope_user_subscription              POST   /myscope/users/:user_id/subscriptions/:id/subscribe_to_email(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"subscribe_to_email", target_type:"users", routing_scope: :myscope }
+    #     unsubscribe_to_email_myscope_user_subscription            POST   /myscope/users/:user_id/subscriptions/:id/unsubscribe_to_email(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"unsubscribe_to_email", target_type:"users", routing_scope: :myscope }
+    #     subscribe_to_optional_target_myscope_user_subscription    POST   /myscope/users/:user_id/subscriptions/:id/subscribe_to_optional_target(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"subscribe_to_optional_target", target_type:"users", routing_scope: :myscope }
+    #     unsubscribe_to_optional_target_myscope_user_subscription  POST   /myscope/users/:user_id/subscriptions/:id/unsubscribe_to_optional_target(.:format)
+    #       { controller:"activity_notification/subscriptions", action:"unsubscribe_to_optional_target", target_type:"users", routing_scope: :myscope }
+    #
     # When you use devise authentication and you want make subscription targets assciated with devise,
     # you can create as follows in your routes:
-    #   notify_to :users, with_devise: :users
+    #   subscribed_by :users, with_devise: :users
     # This with_devise option creates the needed routes assciated with devise authentication:
     #   # Subscription with devise routes
     #     user_subscriptions                                GET    /users/:user_id/subscriptions(.:format)
@@ -151,6 +218,31 @@ module ActionDispatch::Routing
     #     unsubscribe_to_optional_target_user_subscription  POST   /users/:user_id/subscriptions/:id/unsubscribe_to_optional_target(.:format)
     #       { controller:"activity_notification/subscriptions_with_devise", action:"unsubscribe_to_optional_target", target_type:"users", devise_type:"users" }
     #
+    # When you use with_devise option and you want to make simple default routes as follows, you can use devise_default_routes option:
+    #   subscribed_by :users, with_devise: :users, devise_default_routes: true
+    # These with_devise and devise_default_routes options create the needed routes assciated with authenticated devise resource as the default target
+    #   # Subscription with devise routes
+    #     user_subscriptions                                GET    /subscriptions(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"index", target_type:"users", devise_type:"users" }
+    #     user_subscription                                 GET    /subscriptions/:id(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"show", target_type:"users", devise_type:"users" }
+    #     open_all_user_subscriptions                       POST   /subscriptions(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"create", target_type:"users", devise_type:"users" }
+    #     user_subscription                                 DELETE /subscriptions/:id(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"destroy", target_type:"users", devise_type:"users" }
+    #     subscribe_user_subscription                       POST   /subscriptions/:id/subscribe(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"subscribe", target_type:"users", devise_type:"users" }
+    #     unsubscribe_user_subscription                     POST   /subscriptions/:id/unsubscribe(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"unsubscribe", target_type:"users", devise_type:"users" }
+    #     subscribe_to_email_user_subscription              POST   /subscriptions/:id/subscribe_to_email(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"subscribe_to_email", target_type:"users", devise_type:"users" }
+    #     unsubscribe_to_email_user_subscription            POST   /subscriptions/:id/unsubscribe_to_email(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"unsubscribe_to_email", target_type:"users", devise_type:"users" }
+    #     subscribe_to_optional_target_user_subscription    POST   /subscriptions/:id/subscribe_to_optional_target(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"subscribe_to_optional_target", target_type:"users", devise_type:"users" }
+    #     unsubscribe_to_optional_target_user_subscription  POST   /subscriptions/:id/unsubscribe_to_optional_target(.:format)
+    #       { controller:"activity_notification/subscriptions_with_devise", action:"unsubscribe_to_optional_target", target_type:"users", devise_type:"users" }
+    #
     # @example Define subscribed_by in config/routes.rb
     #   subscribed_by :users
     # @example Define subscribed_by with options
@@ -160,29 +252,26 @@ module ActionDispatch::Routing
     #
     # @overload subscribed_by(*resources, *options)
     #   @param          [Symbol]       resources Resources to notify
-    #   @option options [Symbol]       :with_devise (false)          Devise resources name for devise integration. Devise integration will be enabled by this option.
-    #   @option options [String]       :model       (:subscriptions) Model name of subscriptions
-    #   @option options [String]       :controller  ("activity_notification/subscriptions" | activity_notification/subscriptions_with_devise") :controller option as resources routing
-    #   @option options [Symbol]       :as          (nil)            :as option as resources routing
-    #   @option options [Array]        :only        (nil)            :only option as resources routing
-    #   @option options [Array]        :except      (nil)            :except option as resources routing
+    #   @option options [String]       :routing_scope         (nil)            Routing scope for subscription routes
+    #   @option options [Symbol]       :with_devise           (false)          Devise resources name for devise integration. Devise integration will be enabled by this option.
+    #   @option options [Boolean]      :devise_default_routes (false)          Whether you will create routes as device default routes assciated with authenticated devise resource as the default target
+    #   @option options [String]       :model                 (:subscriptions) Model name of subscriptions
+    #   @option options [String]       :controller            ("activity_notification/subscriptions" | activity_notification/subscriptions_with_devise") :controller option as resources routing
+    #   @option options [Symbol]       :as                    (nil)            :as option as resources routing
+    #   @option options [Array]        :only                  (nil)            :only option as resources routing
+    #   @option options [Array]        :except                (nil)            :except option as resources routing
     # @return [ActionDispatch::Routing::Mapper] Routing mapper instance
     def subscribed_by(*resources)
       options = create_options(:subscriptions, resources.extract_options!, [:new, :edit, :update])
 
       resources.each do |target|
-        self.resources target, only: :none do
-          options[:defaults] = { target_type: target.to_s }.merge(options[:devise_defaults])
-          resources_options = options.select { |key, _| [:with_devise, :model, :devise_defaults].exclude? key }
-          self.resources options[:model], resources_options do
-            member do
-              post :subscribe                      unless ignore_path?(:subscribe, options)
-              post :unsubscribe                    unless ignore_path?(:unsubscribe, options)
-              post :subscribe_to_email             unless ignore_path?(:subscribe_to_email, options)
-              post :unsubscribe_to_email           unless ignore_path?(:unsubscribe_to_email, options)
-              post :subscribe_to_optional_target   unless ignore_path?(:subscribe_to_optional_target, options)
-              post :unsubscribe_to_optional_target unless ignore_path?(:unsubscribe_to_optional_target, options)
-            end
+        options[:defaults] = { target_type: target.to_s }.merge(options[:devise_defaults])
+        resources_options = options.select { |key, _| [:with_devise, :devise_default_routes, :model, :devise_defaults].exclude? key }
+        if options[:with_devise].present? && options[:devise_default_routes].present?
+          create_subscription_routes options, resources_options
+        else
+          self.resources target, only: :none do
+            create_subscription_routes options, resources_options
           end
         end
       end
@@ -202,14 +291,16 @@ module ActionDispatch::Routing
         false
       end
 
-      # Create options fo routing
+      # Create options for routing
       # @api private
       # @todo Check resources if it includes target module
       # @todo Check devise configuration in model
       # @todo Support other options like :as, :path_prefix, :path_names ...
       #
       # @param [Symbol] resource Name of the resource model
-      # @return [Boolean] Whether action path is ignored
+      # @param [Hash] options Passed options from notify_to or subscribed_by
+      # @param [Hash] except_actions Actions in [:index, :show, :new, :create, :edit, :update, :destroy] to remove routes
+      # @return [Hash] Options to create routes
       def create_options(resource, options = {}, except_actions = [])
         # Check resources if it includes target module
         resources_name = resource.to_s.pluralize.underscore
@@ -219,6 +310,7 @@ module ActionDispatch::Routing
           options[:as]         ||= resources_name
           # Check devise configuration in model
           options[:devise_defaults] = { devise_type: options[:with_devise].to_s }
+          options[:devise_defaults] = options[:devise_defaults].merge(options.slice(:devise_default_routes))
         else
           options[:controller] ||= "activity_notification/#{resources_name}"
           options[:devise_defaults] = {}
@@ -226,11 +318,48 @@ module ActionDispatch::Routing
         (options[:except] ||= []).concat(except_actions)
         if options[:with_subscription].present?
           options[:subscription_option] = (options[:with_subscription].is_a?(Hash) ? options[:with_subscription] : {})
-                                            .merge(with_devise: options[:with_devise])
+                                            .merge(options.slice(:with_devise, :devise_default_routes, :routing_scope))
         end
         # Support other options like :as, :path_prefix, :path_names ...
         options
       end
 
+      # Create routes for notifications
+      # @api private
+      #
+      # @param [Symbol] resource Name of the resource model
+      # @param [Hash] options Passed options from notify_to
+      # @param [Hash] resources_options Options to send resources method
+      def create_notification_routes(options = {}, resources_options = [])
+        self.resources options[:model], resources_options do
+          collection do
+            post :open_all unless ignore_path?(:open_all, options)
+          end
+          member do
+            get  :move     unless ignore_path?(:move, options)
+            post :open     unless ignore_path?(:open, options)
+          end
+        end
+      end
+
+      # Create routes for subscriptions
+      # @api private
+      #
+      # @param [Symbol] resource Name of the resource model
+      # @param [Hash] options Passed options from subscribed_by
+      # @param [Hash] resources_options Options to send resources method
+      def create_subscription_routes(options = {}, resources_options = [])
+        self.resources options[:model], resources_options do
+          member do
+            post :subscribe                      unless ignore_path?(:subscribe, options)
+            post :unsubscribe                    unless ignore_path?(:unsubscribe, options)
+            post :subscribe_to_email             unless ignore_path?(:subscribe_to_email, options)
+            post :unsubscribe_to_email           unless ignore_path?(:unsubscribe_to_email, options)
+            post :subscribe_to_optional_target   unless ignore_path?(:subscribe_to_optional_target, options)
+            post :unsubscribe_to_optional_target unless ignore_path?(:unsubscribe_to_optional_target, options)
+          end
+        end
+      end
+
   end
 end