activity_notification
activity_notification copied to clipboard
adding nullify_notifiable as option to dependent_notifications
Issue #, if available: https://github.com/simukappu/activity_notification/issues/140
Summary
- Adding a new option to available_dependent_notifications_options called nullify_notifiable.
- Adding a new option to add_destroy_dependency for that key we defined before.
- Defining into the Notifiable module the new function which will update all the generated_notifications_as_notifiable_for(target_type) to notifiable_type and id to nil.
Thank you for your contribution! I think this requested feature is clear, but just wonder what its use cases are. Are there any use cases you would like to nullify notifiable, not deleting notifications? And, I would appreciate it if you could write RSpec tests of this new :nullify_notifiable option.
Let's discuss in #140. Thank you!