Merge pull request #409 from basecamp/read-zero-or-many-notifs-at-once

Read a card should dismiss all notifications

app/controllers/cards/readings_controller.rb

@@ -2,7 +2,7 @@ class Cards::ReadingsController < ApplicationController
   include CardScoped
 
   def create
-    @notification = Current.user.notifications.find_by(card: @card)
-    @notification.read
+    Current.user.notifications.unread.where(card: @card).read_all
+    @notifications = Current.user.notifications.unread.ordered.limit(20)
   end
 end

app/views/cards/readings/create.turbo_stream.erb

@@ -1 +1,3 @@
-<%= turbo_stream.remove @notification %>
+<%= turbo_stream.update "notifications" do %>
+  <%= render collection: @notifications, partial: "notifications/notification" %>
+<% end %>