Fix broken request notifications (#2036)

scope url by organization

Fixes #2034

---------

Co-authored-by: Samuel Oey <samuel.oey@gmail.com>
Co-authored-by: Samuel Oey <so@capinside.com>

app/components/request_notification/request_notification.html.erb

@@ -1,9 +1,9 @@
 <div 
   class="RequestNotification" 
   data-controller="request-notification" 
-  data-request-notification-id-value="<%= request_for_info.id %>" 
   data-request-notification-last-updated-at-value="<%= last_updated_at %>"
   data-request-notification-message-template-value="<%= t('components.request_notification.message_template').to_json %>"
+  data-request-notification-fetch-messages-url-value="<%= notifications_organization_request_path(request_for_info.organization, request_for_info) %>"
   hidden
 >
   <%= c 'notification', show_close: false do %>

app/components/request_notification/request_notification.js

@@ -6,9 +6,9 @@ const POLLING_INTERVAL = 1000 * 30;
 export default class extends Controller {
   static targets = ['text'];
   static values = {
-    id: String,
     lastUpdatedAt: String,
     messageTemplate: String,
+    fetchMessagesUrl: String,
   };
 
   connect() {
@@ -25,7 +25,7 @@ export default class extends Controller {
 
   fetchMessages() {
     Rails.ajax({
-      url: `/requests/${this.idValue}/notifications`,
+      url: this.fetchMessagesUrlValue,
       type: 'GET',
       data: new URLSearchParams({
         last_updated_at: this.lastUpdatedAtValue,

spec/components/request_notification_spec.rb

@@ -6,7 +6,7 @@
   subject { render_inline(described_class.new(**params)) }
 
   let(:params) { { request_for_info: request_for_info } }
-  let(:request_for_info) { build(:request) }
+  let(:request_for_info) { create(:request) }
 
   it { should have_css('.RequestNotification', visible: :hidden) }
 end