Fix creator_id migration

Author: VeljkoMaksimovic

notifications/lib/notifications/api/public_api/create.ex

@@ -15,7 +15,7 @@ defmodule Notifications.Api.PublicApi.Create do
 
     with {:ok, :authorized} <- Auth.can_manage?(user_id, org_id),
          {:ok, :valid} <- Validator.validate(notification),
-         {:ok, n} <- create_notification(org_id, notification) do
+         {:ok, n} <- create_notification(org_id, user_id, notification) do
       Serialization.serialize(n)
     else
       {:error, :permission_denied} ->
@@ -36,13 +36,13 @@ defmodule Notifications.Api.PublicApi.Create do
     end
   end
 
-  def create_notification(org_id, notification) do
+  def create_notification(org_id, creator_id, notification) do
     Repo.transaction(fn ->
       n =
         Models.Notification.new(
           org_id,
           notification.metadata.name,
-          notification.metadata.creator_id,
+          creator_id,
           Notifications.Util.Transforms.encode_spec(notification.spec)
         )
 

notifications/lib/notifications/models/notification.ex

@@ -12,6 +12,7 @@ defmodule Notifications.Models.Notification do
     has_many(:rules, Notifications.Models.Rule, on_delete: :delete_all)
 
     field(:org_id, :binary_id)
+    field(:creator_id, :binary_id)
     field(:name, :string)
     field(:spec, :map)
 

notifications/priv/repo/migrations/20250804163140_add_creator_id_to_notifications.exs

@@ -0,0 +1,9 @@
+defmodule Notifications.Repo.Migrations.AddCreatorIdToNotifications do
+  use Ecto.Migration
+
+  def change do
+    alter table(:notifications) do
+      add :creator_id, :binary_id, null: true
+    end
+  end
+end