Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: e36f8b8b3c91b22b1f7a4af421e421e7654a44ed https://github.com/openSUSE/open-build-service/commit/e36f8b8b3c91b22b1f7a4a... Author: Dani Donisa <dani.doni@gmail.com> Date: 2024-05-16 (Thu, 16 May 2024) Changed paths: M src/api/app/components/notification_filter_component.rb M src/api/app/controllers/person/notifications_controller.rb M src/api/app/controllers/webui/users/notifications_controller.rb M src/api/app/jobs/cleanup_notifications_job.rb M src/api/app/models/notification.rb M src/api/app/models/user.rb R src/api/app/queries/notifications_finder.rb M src/api/app/services/notification_service/web_channel.rb M src/api/spec/db/data/remove_obsolete_notifications_spec.rb Log Message: ----------- Remove NotificationsFinder It's just a wrapper around Notification scopes, with the single exception of the `for_notifiable_type` that is a bit more complex, but doesn't pay off having a whole class just for that. Commit: 1729d8ad489f1260a7434777e6b3c4a135afd17a https://github.com/openSUSE/open-build-service/commit/1729d8ad489f1260a74347... Author: Dani Donisa <dani.doni@gmail.com> Date: 2024-05-16 (Thu, 16 May 2024) Changed paths: M src/api/app/components/notification_filter_component.rb M src/api/app/controllers/person/notifications_controller.rb M src/api/app/controllers/webui/users/notifications_controller.rb M src/api/app/models/notification.rb Log Message: ----------- Make the for_notifiable_type scope simpler Commit: c2bd27a9cc542bdb77480d240ec3a063c37e74e2 https://github.com/openSUSE/open-build-service/commit/c2bd27a9cc542bdb77480d... Author: Dani Donisa <dani.doni@gmail.com> Date: 2024-05-16 (Thu, 16 May 2024) Changed paths: M src/api/app/components/notification_filter_component.rb M src/api/app/controllers/person/notifications_controller.rb M src/api/app/controllers/webui/users/notifications_controller.rb M src/api/app/jobs/cleanup_notifications_job.rb M src/api/app/models/notification.rb M src/api/app/models/user.rb R src/api/app/queries/notifications_finder.rb M src/api/app/services/notification_service/web_channel.rb M src/api/spec/db/data/remove_obsolete_notifications_spec.rb Log Message: ----------- Merge pull request #16143 from danidoni/remove-notifications-finder Remove NotificationsFinder Compare: https://github.com/openSUSE/open-build-service/compare/e9a35f2adb15...c2bd27... To unsubscribe from these emails, change your notification settings at https://github.com/openSUSE/open-build-service/settings/notifications