Remove redundant code

Author: KyleMacPherson

app/controllers/jobseekers/base_controller.rb

@@ -11,7 +11,7 @@ def mark_notification_as_read_if_present
       return if params[:notification_id].blank?
 
       notification = current_jobseeker.notifications.find_by(id: params[:notification_id])
-      notification&.mark_as_read!
+      notification&.mark_as_read
     end
   end
 end

app/controllers/publishers/base_controller.rb

@@ -22,12 +22,9 @@ def check_ats_interstitial_acknowledged
 
     def mark_notification_as_read_if_present
       return if params[:notification_id].blank?
-      return if current_publisher.blank?
 
       notification = current_publisher.notifications.find_by(id: params[:notification_id])
-      notification&.mark_as_read!
-    rescue StandardError => e
-      Rails.logger.error("Failed to mark notification as read: #{e.message}")
+      notification&.mark_as_read
     end
   end
 end