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

Merge pull request #1198 from Retrospring/fix/notification-fragment-cache 

Fix incorrect caching method used for notifications dropdown
This commit is contained in:
Karina Kwiatek 2023-05-07 16:59:09 +02:00 committed by GitHub
parent cd49558b81 15405e41c5
commit ff7c5bb967
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/views/navigation/_desktop.html.haml
View File

@ -31,7 +31,7 @@
%span.badge= notification_count %span.badge= notification_count
.dropdown-menu.dropdown-menu-end.notification-dropdown .dropdown-menu.dropdown-menu-end.notification-dropdown
%turbo-frame#notifications-dropdown-list %turbo-frame#notifications-dropdown-list
- Rails.cache.fetch current_user.notification_dropdown_cache_key do - cache current_user.notification_dropdown_cache_key do
- notifications = Notification.for(current_user).where(new: true).includes([:target]).limit(4) - notifications = Notification.for(current_user).where(new: true).includes([:target]).limit(4)
= render "navigation/dropdown/notifications", notifications:, size: "desktop" = render "navigation/dropdown/notifications", notifications:, size: "desktop"
%li.nav-item.d-none.d-sm-block{ data: { bs_toggle: 'tooltip', bs_placement: 'bottom' }, title: t('.ask_question') } %li.nav-item.d-none.d-sm-block{ data: { bs_toggle: 'tooltip', bs_placement: 'bottom' }, title: t('.ask_question') }