activity_notification

activity_notification copied to clipboard

### Steps to reproduce In NotificationApi#notify the call to targets.blank? (https://github.com/simukappu/activity_notification/blob/master/lib/activity_notification/apis/notification_api.rb#L231) loads all of the targets, for a large collection, this loads all the records into memory and it seems...

markedmondson

I would like to expire old notifications.. is the best way to do it to just something like: `ActivityNotification.where("created_at < ?", Time.now-6.months).destroy_all` Is this the right way to do it?...

netpenthe

Bulk destroy notifications

2

### Problem or use case: Couldn't find a way to delete more than one notification for a target. While the `destroy` API provides functionality to destroy a single notification, I...

Mack241

### Problem or use case Gemspec disallows Rails 7.1 usage. ### Expected solution Allow rails 7.1 dependencies - not sure if additional work is required.

jprosevear

After adding gem to project get: Undefined method `protect_from_forgery' for ActivityNotification::SubscriptionsApiController:Class (NoMethodError)

1

### Steps to reproduce 1. Add `gem 'activity_notification'` to Gemfile 2. Run `bundle install` 3. Start rails 4. Get error ### System configuration **activity_notification gem version**: 2.2.3 **Rails version**: 6.1.4.4...

0vermore

With 20k+ notification the index request time out with code 504

1

### Steps to reproduce Have more than 20k notifications ### Expected behavior Get request works fine ### Actual behavior Request timeout ### Gem activity notification without number **Rails version**: 6.0.1...

NehalAbdelmegeed

For #173. I would respectfully suggest that 6.0 and lower be dropped soon. Rails 7.2 is in beta and 8 is in heavy development. This would also drop dynamoid support.

jprosevear

Rails 7.2 support

2

jprosevear

Rails 7.2 support

1

### Problem or use case Gemspec does not allow Rails 7.2 use. ### Expected solution Allow Rails 7.2 dependencies.

simukappu