FreakU
/
Retrospring
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1203 from Retrospring/fix/notification-bugs 

Notification dropdown fixes
This commit is contained in:
Karina Kwiatek 2023-05-08 19:16:33 +02:00 committed by GitHub
parent 9e9b651189 2db4ce38c9
commit 333af458db
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/controllers/notifications_controller.rb
View File

@ -27,6 +27,7 @@ class NotificationsController < ApplicationController
def read
current_user.notifications.where(new: true).update_all(new: false) # rubocop:disable Rails/SkipsModelValidations
current_user.touch(:notifications_updated_at)
respond_to do |format|
format.turbo_stream do

2
app/views/notifications/type/_nilclass.html.haml Normal file
View File

@ -0,0 +1,2 @@
-# This is a stub to prevent errors on broken notifications on content being asynchronously destroyed.
- logger.error "Notification ##{notification.id} has a target which doesn't exist. (Target: #{notification.target_type}##{notification.target_id})"