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
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
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
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