feat(front): display who was last to modify notification (#505)

## 📝 Description

renderedtext/project-tasks#2659

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update

Author: VeljkoMaksimovic

front/lib/front_web/controllers/notifications_controller.ex

@@ -4,11 +4,7 @@ defmodule FrontWeb.NotificationsController do
 
   alias Front.Async
   alias Front.Audit
-
-  alias Front.Models.{
-    Notification,
-    Organization
-  }
+  alias Front.Models.{Notification, Organization, User}
 
   plug(FrontWeb.Plugs.FetchPermissions, scope: "org")
   plug(FrontWeb.Plugs.PageAccess, permissions: "organization.view")
@@ -25,6 +21,7 @@ defmodule FrontWeb.NotificationsController do
 
       {:ok, organization} = Async.await(fetch_organization)
       {:ok, notifications} = Async.await(fetch_notifications)
+      notifications = add_user_data_to_notifications(notifications)
 
       render(
         conn,
@@ -298,6 +295,27 @@ defmodule FrontWeb.NotificationsController do
     |> render("404.html")
   end
 
+  defp add_user_data_to_notifications(notifications) when is_list(notifications) do
+    import Enum
+
+    creators =
+      notifications
+      |> map(& &1.metadata.creator_id)
+      |> reject(&(&1 == ""))
+      |> uniq()
+      |> User.find_many()
+
+    map(notifications, fn n ->
+      if n.metadata.creator_id != "" do
+        default_username = Application.get_env(:front, :default_user_name)
+        creator = find(creators, &(&1.id == n.metadata.creator_id)) || %{name: default_username}
+        %{n | metadata: Map.put(n.metadata, :creator, creator)}
+      else
+        n
+      end
+    end)
+  end
+
   def get_rule_identifiers(params) do
     params
     |> Map.keys()

front/lib/front_web/templates/notifications/_notifications.html.eex

@@ -1,6 +1,11 @@
 <%= Enum.map(@notifications, fn notification -> %>
   <div class="bg-white shadow-1 br3 pa3 mv3">
-        <div class="f4 f3-m b mb2"><%= notification.metadata.name %></div>
+        <div class="flex items-center justify-between mb2">
+            <div class="f4 f3-m b"><%= notification.metadata.name %></div>
+            <%= if Map.has_key?(notification.metadata, :creator) && Map.has_key?(notification.metadata.creator, :name) do %>
+              <div class="f6 black-50 tr">last modified by <%= notification.metadata.creator.name %></div>
+            <% end %>
+        </div>
 
         <div class="overflow-auto nowrap">
           <%= Enum.map(notification.spec.rules, fn rule -> %>

notifications/lib/notifications/api/public_api/serialization.ex

@@ -15,6 +15,7 @@ defmodule Notifications.Api.PublicApi.Serialization do
     Notification.Metadata.new(
       name: notification.name,
       id: notification.id,
+      creator_id: notification.creator_id,
       create_time: unix_timestamp(notification.inserted_at),
       update_time: unix_timestamp(notification.updated_at)
     )