Merge pull request #14690 from danidoni/fix/reuse-existing-event-subs…

…criptions-to-notify-scms-back Reuse existing event subscription to notify SCMs back
openSUSE · Jul 24, 2023 · 9035d4e · 9035d4e
2 parents d208d5f + ead8164
commit 9035d4e
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/api/app/services/workflows/scm_event_subscription_creator.rb b/src/api/app/services/workflows/scm_event_subscription_creator.rb
@@ -16,9 +16,9 @@ def call
                                              channel: 'scm',
                                              enabled: true,
                                              token: @token,
-                                             package: @package,
-                                             workflow_run: @workflow_run).tap do |subscription|
-          subscription.update!(payload: @scm_webhook.payload) # The payload is updated regardless of whether the subscription already existed or not.
+                                             package: @package).tap do |subscription|
+          # Set payload and workflow_run regardless of whether the subscription already existed or not
+          subscription.update!(workflow_run: @workflow_run, payload: @scm_webhook.payload)
         end
       end
     end