chore: release version v1.0.14

Show error notification if no character tracked

.github/FUNDING.yml

@@ -0,0 +1,15 @@
+# These are supported funding model platforms
+
+github: # Replace with up to 4 GitHub Sponsors-enabled usernames e.g., [user1, user2]
+patreon: WandererLtd
+open_collective: # Replace with a single Open Collective username
+ko_fi: # Replace with a single Ko-fi username
+tidelift: # Replace with a single Tidelift platform-name/package-name e.g., npm/babel
+community_bridge: # Replace with a single Community Bridge project-name e.g., cloud-foundry
+liberapay: # Replace with a single Liberapay username
+issuehunt: # Replace with a single IssueHunt username
+lfx_crowdfunding: # Replace with a single LFX Crowdfunding project-name e.g., cloud-foundry
+polar: # Replace with a single Polar username
+buy_me_a_coffee: # Replace with a single Buy Me a Coffee username
+thanks_dev: # Replace with a single thanks.dev username
+custom: # Replace with up to 4 custom sponsorship URLs e.g., ['link1', 'link2']

CHANGELOG.md

@@ -2,6 +2,43 @@
 
 <!-- changelog -->
 
+## [v1.0.14](https://github.com/wanderer-industries/wanderer/compare/v1.0.13...v1.0.14) (2024-09-21)
+
+
+
+
+## [v1.0.13](https://github.com/wanderer-industries/wanderer/compare/v1.0.12...v1.0.13) (2024-09-21)
+
+
+
+
+### Bug Fixes:
+
+* tracking: Ensure user has at least one character tracked to work with map
+
+## [v1.0.12](https://github.com/wanderer-industries/wanderer/compare/v1.0.11...v1.0.12) (2024-09-20)
+
+
+
+
+### Bug Fixes:
+
+* audit: Hide character for non-character map activities
+
+## [v1.0.11](https://github.com/wanderer-industries/wanderer/compare/v1.0.10...v1.0.11) (2024-09-20)
+
+
+
+
+## [v1.0.10](https://github.com/wanderer-industries/wanderer/compare/v1.0.9...v1.0.10) (2024-09-19)
+
+
+
+
+### Bug Fixes:
+
+* signatures: Fix update signatures error if no character tracked on map
+
 ## [v1.0.9](https://github.com/wanderer-industries/wanderer/compare/v1.0.8...v1.0.9) (2024-09-19)
 
 

config/runtime.exs

@@ -53,6 +53,15 @@ map_subscriptions_enabled =
   |> get_var_from_path_or_env("WANDERER_MAP_SUBSCRIPTIONS_ENABLED", "false")
   |> String.to_existing_atom()
 
+
+map_subscription_characters_limit =
+  config_dir
+  |> get_int_from_path_or_env("WANDERER_MAP_SUBSCRIPTION_CHARACTERS_LIMIT", 100)
+
+map_subscription_hubs_limit =
+  config_dir
+  |> get_int_from_path_or_env("WANDERER_MAP_SUBSCRIPTION_HUBS_LIMIT", 10)
+
 wallet_tracking_enabled =
   config_dir
   |> get_var_from_path_or_env("WANDERER_WALLET_TRACKING_ENABLED", "false")
@@ -77,7 +86,7 @@ config :wanderer_app,
   wallet_tracking_enabled: wallet_tracking_enabled,
   subscription_settings: %{
     plans: [
-      %{id: "alpha", characters_limit: 100, hubs_limit: 10, base_price: 0, monthly_discount: 0},
+      %{id: "alpha", characters_limit: map_subscription_characters_limit, hubs_limit: map_subscription_hubs_limit, base_price: 0, monthly_discount: 0},
       %{
         id: "omega",
         characters_limit: 300,

lib/wanderer_app_web/components/alerts.ex

@@ -51,7 +51,7 @@ defmodule WandererAppWeb.Alerts do
   def delayed_fade_out_flash() do
     JS.hide(
       transition:
-        {"transition-opacity ease-out delay-2000 duration-1000", "opacity-100", "opacity-0"},
+        {"transition-opacity ease-out delay-5000 duration-6000", "opacity-100", "opacity-0"},
       time: 6000
     )
     |> JS.push("lv:clear-flash")

lib/wanderer_app_web/components/layouts/root.html.heex

@@ -42,27 +42,30 @@
       integrity={integrity_hash("https://unpkg.com/react-dom@18/umd/react-dom.production.min.js")}
     >
     </script>
+
+    <script defer phx-track-static type="module" src={~p"/assets/app.js"} crossorigin="anonymous">
+    </script>
+
     <!-- Appzi: Capture Insightful Feedback -->
-    <script async src="https://w.appzi.io/w.js?token=yddv0">
+    <script defer src="https://w.appzi.io/w.js?token=yddv0">
     </script>
     <!-- End Appzi -->
 
     <!-- Google tag (gtag.js) -->
     <script
-      async
+      defer
       src="https://www.googletagmanager.com/gtag/js?id=G-61PHLLS0LD"
       crossorigin="anonymous"
     >
     </script>
-    <script>
+    <script defer>
       window.dataLayer = window.dataLayer || [];
       function gtag(){dataLayer.push(arguments);}
       gtag('js', new Date());
 
       gtag('config', 'G-61PHLLS0LD');
     </script>
-    <script defer phx-track-static type="module" src={~p"/assets/app.js"} crossorigin="anonymous">
-    </script>
+
   </head>
   <body>
     <%= @inner_content %>

lib/wanderer_app_web/components/user_activity.ex

@@ -46,7 +46,7 @@ defmodule WandererAppWeb.UserActivity do
             <.local_time id={@activity.id} at={@activity.inserted_at} />
           </span>
         </p>
-        <p class="flex shrink-0 items-center space-x-1 min-w-[200px]">
+        <p :if={not is_nil(@activity.character)} class="flex shrink-0 items-center space-x-1 min-w-[200px]">
           <.character_item character={@activity.character} />
         </p>
       </div>

lib/wanderer_app_web/live/maps/map_live.ex

@@ -386,6 +386,13 @@ defmodule WandererAppWeb.MapLive do
                   "One of your characters has expired token. Please refresh it on characters page."
                 )
 
+              :empty_tracked_characters ->
+                socket
+                |> put_flash(
+                  :info,
+                  "You should enable tracking for at least one character to work with map."
+                )
+
               :map_character_limit ->
                 socket
                 |> put_flash(
@@ -400,7 +407,7 @@ defmodule WandererAppWeb.MapLive do
 
         {:noreply,
          socket
-         |> assign(map_loaded?: true, user_characters: user_character_eve_ids)
+         |> assign(map_loaded?: true, user_characters: user_character_eve_ids, has_tracked_characters?: _has_tracked_characters?(user_character_eve_ids))
          |> _push_map_event("init", initial_data)
          |> push_event("js-exec", %{
            to: "#map-loader",
@@ -470,16 +477,24 @@ defmodule WandererAppWeb.MapLive do
   end
 
   @impl true
-  def handle_event("manual_add_system", %{"coordinates" => coordinates} = _event, socket) do
-    case _check_user_permissions(socket, :add_system) do
+  def handle_event("manual_add_system", %{"coordinates" => coordinates} = _event, %{assigns: %{has_tracked_characters?: has_tracked_characters?}} = socket) do
+    case has_tracked_characters? do
       true ->
-        {:noreply,
-         socket
-         |> assign(coordinates: coordinates)
-         |> push_patch(to: ~p"/#{socket.assigns.map_slug}/add-system")}
+        case _check_user_permissions(socket, :add_system) do
+          true ->
+            {:noreply,
+            socket
+            |> assign(coordinates: coordinates)
+            |> push_patch(to: ~p"/#{socket.assigns.map_slug}/add-system")}
 
+          _ ->
+            {:noreply, socket}
+        end
       _ ->
-        {:noreply, socket}
+        {:noreply, socket |> put_flash(
+          :error,
+          "You should enable tracking for at least one character to add system."
+        )}
     end
   end
 
@@ -487,157 +502,189 @@ defmodule WandererAppWeb.MapLive do
   def handle_event(
         "manual_add_connection",
         %{"source" => solar_system_source_id, "target" => solar_system_target_id} = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :add_connection) do
-      true ->
-        map_id =
-          socket
-          |> map_id()
+          case has_tracked_characters? do
+            true ->
+              case _check_user_permissions(socket, :add_connection) do
+                true ->
+                  map_id =
+                    socket
+                    |> map_id()
 
-        map_id
-        |> WandererApp.Map.Server.add_connection(%{
-          solar_system_source_id: solar_system_source_id |> String.to_integer(),
-          solar_system_target_id: solar_system_target_id |> String.to_integer()
-        })
+                  map_id
+                  |> WandererApp.Map.Server.add_connection(%{
+                    solar_system_source_id: solar_system_source_id |> String.to_integer(),
+                    solar_system_target_id: solar_system_target_id |> String.to_integer()
+                  })
 
-        :telemetry.execute([:wanderer_app, :map, :connection, :add], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
-          solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
-        })
+                  :telemetry.execute([:wanderer_app, :map, :connection, :add], %{count: 1}, %{
+                    character_id: tracked_character_ids |> List.first(),
+                    user_id: current_user.id,
+                    map_id: map_id,
+                    solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
+                    solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
+                  })
 
-        {:noreply, socket}
+                  {:noreply, socket}
 
-      _ ->
-        {:noreply, socket}
-    end
+                _ ->
+                  {:noreply, socket}
+              end
+            _ ->
+              {:noreply, socket |> put_flash(
+                :error,
+                "You should enable tracking for at least one character to add connection."
+              )}
+          end
   end
 
   @impl true
   def handle_event(
         "add_hub",
         %{"system_id" => solar_system_id} = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
-      true ->
-        map_id =
-          socket
-          |> map_id()
+        case has_tracked_characters? do
+          true ->
+            case _check_user_permissions(socket, :update_system) do
+              true ->
+                map_id =
+                  socket
+                  |> map_id()
 
-        :telemetry.execute([:wanderer_app, :map, :hub, :add], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_id: solar_system_id
-        })
+                :telemetry.execute([:wanderer_app, :map, :hub, :add], %{count: 1}, %{
+                  character_id: tracked_character_ids |> List.first(),
+                  user_id: current_user.id,
+                  map_id: map_id,
+                  solar_system_id: solar_system_id
+                })
 
-        map_id
-        |> WandererApp.Map.Server.add_hub(%{
-          solar_system_id: solar_system_id
-        })
+                map_id
+                |> WandererApp.Map.Server.add_hub(%{
+                  solar_system_id: solar_system_id
+                })
 
-        {:noreply, socket}
+                {:noreply, socket}
 
-      _ ->
-        {:noreply, socket}
-    end
+              _ ->
+                {:noreply, socket}
+            end
+          _ ->
+            {:noreply, socket |> put_flash(
+              :error,
+              "You should enable tracking for at least one character to add hub."
+            )}
+        end
   end
 
   @impl true
   def handle_event(
         "delete_hub",
         %{"system_id" => solar_system_id} = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
-      true ->
-        map_id =
-          socket
-          |> map_id()
+        case has_tracked_characters? do
+          true ->
+            case _check_user_permissions(socket, :update_system) do
+              true ->
+                map_id =
+                  socket
+                  |> map_id()
 
-        :telemetry.execute([:wanderer_app, :map, :hub, :remove], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_id: solar_system_id
-        })
+                :telemetry.execute([:wanderer_app, :map, :hub, :remove], %{count: 1}, %{
+                  character_id: tracked_character_ids |> List.first(),
+                  user_id: current_user.id,
+                  map_id: map_id,
+                  solar_system_id: solar_system_id
+                })
 
-        map_id
-        |> WandererApp.Map.Server.remove_hub(%{
-          solar_system_id: solar_system_id
-        })
+                map_id
+                |> WandererApp.Map.Server.remove_hub(%{
+                  solar_system_id: solar_system_id
+                })
 
-        {:noreply, socket}
+                {:noreply, socket}
 
-      _ ->
-        {:noreply, socket}
-    end
+              _ ->
+                {:noreply, socket}
+            end
+          _ ->
+            {:noreply, socket |> put_flash(
+              :error,
+              "You should enable tracking for at least one character to remove hub."
+            )}
+        end
   end
 
   @impl true
   def handle_event(
         "update_system_" <> param,
         %{"system_id" => solar_system_id, "value" => value} = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
-      true ->
-        method_atom =
-          case param do
-            "name" -> :update_system_name
-            "description" -> :update_system_description
-            "labels" -> :update_system_labels
-            "locked" -> :update_system_locked
-            "tag" -> :update_system_tag
-            "status" -> :update_system_status
-            _ -> nil
-          end
+        case has_tracked_characters? do
+          true ->
+            case _check_user_permissions(socket, :update_system) do
+              true ->
+                method_atom =
+                  case param do
+                    "name" -> :update_system_name
+                    "description" -> :update_system_description
+                    "labels" -> :update_system_labels
+                    "locked" -> :update_system_locked
+                    "tag" -> :update_system_tag
+                    "status" -> :update_system_status
+                    _ -> nil
+                  end
 
-        key_atom =
-          case param do
-            "name" -> :name
-            "description" -> :description
-            "labels" -> :labels
-            "locked" -> :locked
-            "tag" -> :tag
-            "status" -> :status
-            _ -> :none
-          end
+                key_atom =
+                  case param do
+                    "name" -> :name
+                    "description" -> :description
+                    "labels" -> :labels
+                    "locked" -> :locked
+                    "tag" -> :tag
+                    "status" -> :status
+                    _ -> :none
+                  end
 
-        map_id =
-          socket
-          |> map_id()
+                map_id =
+                  socket
+                  |> map_id()
 
-        :telemetry.execute([:wanderer_app, :map, :system, :update], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_id: "#{solar_system_id}" |> String.to_integer(),
-          key: key_atom,
-          value: value
-        })
+                :telemetry.execute([:wanderer_app, :map, :system, :update], %{count: 1}, %{
+                  character_id: tracked_character_ids |> List.first(),
+                  user_id: current_user.id,
+                  map_id: map_id,
+                  solar_system_id: "#{solar_system_id}" |> String.to_integer(),
+                  key: key_atom,
+                  value: value
+                })
 
-        apply(WandererApp.Map.Server, method_atom, [
-          map_id,
-          %{
-            solar_system_id: "#{solar_system_id}" |> String.to_integer()
-          }
-          |> Map.put_new(key_atom, value)
-        ])
+                apply(WandererApp.Map.Server, method_atom, [
+                  map_id,
+                  %{
+                    solar_system_id: "#{solar_system_id}" |> String.to_integer()
+                  }
+                  |> Map.put_new(key_atom, value)
+                ])
 
-        {:noreply, socket}
+                {:noreply, socket}
 
-      _ ->
-        {:noreply, socket}
-    end
+              _ ->
+                {:noreply, socket}
+            end
+          _ ->
+            {:noreply, socket |> put_flash(
+              :error,
+              "You should enable tracking for at least one character to update system."
+            )}
+        end
   end
 
   @impl true
@@ -648,57 +695,65 @@ defmodule WandererAppWeb.MapLive do
           "target" => solar_system_target_id,
           "value" => value
         } = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
-      true ->
-        method_atom =
-          case param do
-            "time_status" -> :update_connection_time_status
-            "mass_status" -> :update_connection_mass_status
-            "ship_size_type" -> :update_connection_ship_size_type
-            "locked" -> :update_connection_locked
-            _ -> nil
-          end
+        case has_tracked_characters? do
+          true ->
+            case _check_user_permissions(socket, :update_system) do
+              true ->
+                method_atom =
+                  case param do
+                    "time_status" -> :update_connection_time_status
+                    "mass_status" -> :update_connection_mass_status
+                    "ship_size_type" -> :update_connection_ship_size_type
+                    "locked" -> :update_connection_locked
+                    _ -> nil
+                  end
 
-        key_atom =
-          case param do
-            "time_status" -> :time_status
-            "mass_status" -> :mass_status
-            "ship_size_type" -> :ship_size_type
-            "locked" -> :locked
-            _ -> nil
-          end
+                key_atom =
+                  case param do
+                    "time_status" -> :time_status
+                    "mass_status" -> :mass_status
+                    "ship_size_type" -> :ship_size_type
+                    "locked" -> :locked
+                    _ -> nil
+                  end
 
-        map_id =
-          socket
-          |> map_id()
+                map_id =
+                  socket
+                  |> map_id()
 
-        :telemetry.execute([:wanderer_app, :map, :connection, :update], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
-          solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer(),
-          key: key_atom,
-          value: value
-        })
+                :telemetry.execute([:wanderer_app, :map, :connection, :update], %{count: 1}, %{
+                  character_id: tracked_character_ids |> List.first(),
+                  user_id: current_user.id,
+                  map_id: map_id,
+                  solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
+                  solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer(),
+                  key: key_atom,
+                  value: value
+                })
 
-        apply(WandererApp.Map.Server, method_atom, [
-          map_id,
-          %{
-            solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
-            solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
-          }
-          |> Map.put_new(key_atom, value)
-        ])
+                apply(WandererApp.Map.Server, method_atom, [
+                  map_id,
+                  %{
+                    solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
+                    solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
+                  }
+                  |> Map.put_new(key_atom, value)
+                ])
 
-        {:noreply, socket}
+                {:noreply, socket}
 
-      _ ->
-        {:noreply, socket}
-    end
+              _ ->
+                {:noreply, socket}
+            end
+          _ ->
+            {:noreply, socket |> put_flash(
+              :error,
+              "You should enable tracking for at least one character to update connection."
+            )}
+        end
   end
 
   @impl true
@@ -724,47 +779,55 @@ defmodule WandererAppWeb.MapLive do
             first_character_eve_id =
               Map.get(socket.assigns, :user_characters, []) |> List.first()
 
-            added_signatures =
-              added_signatures
-              |> _parse_signatures(first_character_eve_id, system.id)
+            case not is_nil(first_character_eve_id) do
+              true ->
+                added_signatures =
+                  added_signatures
+                  |> _parse_signatures(first_character_eve_id, system.id)
 
-            updated_signatures =
-              updated_signatures
-              |> _parse_signatures(first_character_eve_id, system.id)
+                updated_signatures =
+                  updated_signatures
+                  |> _parse_signatures(first_character_eve_id, system.id)
 
-            updated_signatures_eve_ids =
-              updated_signatures
-              |> Enum.map(fn s -> s.eve_id end)
+                updated_signatures_eve_ids =
+                  updated_signatures
+                  |> Enum.map(fn s -> s.eve_id end)
 
-            removed_signatures_eve_ids =
-              removed_signatures
-              |> _parse_signatures(first_character_eve_id, system.id)
-              |> Enum.map(fn s -> s.eve_id end)
+                removed_signatures_eve_ids =
+                  removed_signatures
+                  |> _parse_signatures(first_character_eve_id, system.id)
+                  |> Enum.map(fn s -> s.eve_id end)
 
-            WandererApp.Api.MapSystemSignature.by_system_id!(system.id)
-            |> Enum.filter(fn s -> s.eve_id in removed_signatures_eve_ids end)
-            |> Enum.each(fn s ->
-              s
-              |> Ash.destroy!()
-            end)
+                WandererApp.Api.MapSystemSignature.by_system_id!(system.id)
+                |> Enum.filter(fn s -> s.eve_id in removed_signatures_eve_ids end)
+                |> Enum.each(fn s ->
+                  s
+                  |> Ash.destroy!()
+                end)
 
-            WandererApp.Api.MapSystemSignature.by_system_id!(system.id)
-            |> Enum.filter(fn s -> s.eve_id in updated_signatures_eve_ids end)
-            |> Enum.each(fn s ->
-              updated = updated_signatures |> Enum.find(fn u -> u.eve_id == s.eve_id end)
+                WandererApp.Api.MapSystemSignature.by_system_id!(system.id)
+                |> Enum.filter(fn s -> s.eve_id in updated_signatures_eve_ids end)
+                |> Enum.each(fn s ->
+                  updated = updated_signatures |> Enum.find(fn u -> u.eve_id == s.eve_id end)
 
-              if not is_nil(updated) do
-                s
-                |> WandererApp.Api.MapSystemSignature.update(updated)
-              end
-            end)
+                  if not is_nil(updated) do
+                    s
+                    |> WandererApp.Api.MapSystemSignature.update(updated)
+                  end
+                end)
 
-            added_signatures
-            |> Enum.map(fn s ->
-              s |> WandererApp.Api.MapSystemSignature.create!()
-            end)
+                added_signatures
+                |> Enum.map(fn s ->
+                  s |> WandererApp.Api.MapSystemSignature.create!()
+                end)
 
-            {:reply, %{signatures: _get_system_signatures(system.id)}, socket}
+                {:reply, %{signatures: _get_system_signatures(system.id)}, socket}
+              _ ->
+                {:reply, %{signatures: []}, socket |> put_flash(
+                  :error,
+                  "You should enable tracking for at least one character to work with signatures."
+                )}
+            end
 
           _ ->
             {:noreply, socket}
@@ -883,18 +946,26 @@ defmodule WandererAppWeb.MapLive do
   def handle_event(
         "update_system_position",
         position,
-        socket
+        %{assigns: %{has_tracked_characters?: has_tracked_characters?}} = socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
+    case has_tracked_characters? do
       true ->
-        socket
-        |> map_id()
-        |> _update_system_position(position)
+        case _check_user_permissions(socket, :update_system) do
+          true ->
+            socket
+            |> map_id()
+            |> _update_system_position(position)
 
-        {:noreply, socket}
+            {:noreply, socket}
 
+          _ ->
+            {:noreply, socket}
+        end
       _ ->
-        {:noreply, socket}
+        {:noreply, socket |> put_flash(
+          :error,
+          "You should enable tracking for at least one character to update system."
+        )}
     end
   end
 
@@ -902,18 +973,26 @@ defmodule WandererAppWeb.MapLive do
   def handle_event(
         "update_system_positions",
         positions,
-        socket
+        %{assigns: %{has_tracked_characters?: has_tracked_characters?}} = socket
       ) do
-    case _check_user_permissions(socket, :update_system) do
+    case has_tracked_characters? do
       true ->
-        socket
-        |> map_id()
-        |> _update_system_positions(positions)
+        case _check_user_permissions(socket, :update_system) do
+            true ->
+              socket
+              |> map_id()
+              |> _update_system_positions(positions)
 
-        {:noreply, socket}
+              {:noreply, socket}
 
+            _ ->
+              {:noreply, socket}
+          end
       _ ->
-        {:noreply, socket}
+        {:noreply, socket |> put_flash(
+          :error,
+          "You should enable tracking for at least one character to update systems."
+        )}
     end
   end
 
@@ -921,58 +1000,76 @@ defmodule WandererAppWeb.MapLive do
   def handle_event(
         "delete_systems",
         solar_system_ids,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :delete_system) do
+    case has_tracked_characters? do
       true ->
-        socket
-        |> map_id()
-        |> WandererApp.Map.Server.delete_systems(
-          solar_system_ids |> Enum.map(&String.to_integer/1),
-          current_user.id,
-          tracked_character_ids |> List.first()
-        )
+        case _check_user_permissions(socket, :delete_system) do
+          true ->
+            socket
+            |> map_id()
+            |> WandererApp.Map.Server.delete_systems(
+              solar_system_ids |> Enum.map(&String.to_integer/1),
+              current_user.id,
+              tracked_character_ids |> List.first()
+            )
 
-        {:noreply, socket}
+            {:noreply, socket}
 
+          _ ->
+            {:noreply, socket}
+        end
       _ ->
-        {:noreply, socket}
+        {:noreply, socket |> put_flash(
+          :error,
+          "You should enable tracking for at least one character to delete systems."
+        )}
     end
+
   end
 
   @impl true
   def handle_event(
         "manual_delete_connection",
         %{"source" => solar_system_source_id, "target" => solar_system_target_id} = _event,
-        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids}} =
+        %{assigns: %{current_user: current_user, tracked_character_ids: tracked_character_ids, has_tracked_characters?: has_tracked_characters?}} =
           socket
       ) do
-    case _check_user_permissions(socket, :delete_connection) do
+    case has_tracked_characters? do
       true ->
-        map_id =
-          socket
-          |> map_id()
+        case _check_user_permissions(socket, :delete_connection) do
+          true ->
+            map_id =
+              socket
+              |> map_id()
 
-        map_id
-        |> WandererApp.Map.Server.delete_connection(%{
-          solar_system_source_id: solar_system_source_id |> String.to_integer(),
-          solar_system_target_id: solar_system_target_id |> String.to_integer()
-        })
+            map_id
+            |> WandererApp.Map.Server.delete_connection(%{
+              solar_system_source_id: solar_system_source_id |> String.to_integer(),
+              solar_system_target_id: solar_system_target_id |> String.to_integer()
+            })
 
-        :telemetry.execute([:wanderer_app, :map, :connection, :remove], %{count: 1}, %{
-          character_id: tracked_character_ids |> List.first(),
-          user_id: current_user.id,
-          map_id: map_id,
-          solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
-          solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
-        })
+            :telemetry.execute([:wanderer_app, :map, :connection, :remove], %{count: 1}, %{
+              character_id: tracked_character_ids |> List.first(),
+              user_id: current_user.id,
+              map_id: map_id,
+              solar_system_source_id: "#{solar_system_source_id}" |> String.to_integer(),
+              solar_system_target_id: "#{solar_system_target_id}" |> String.to_integer()
+            })
 
-        {:noreply, socket}
+            {:noreply, socket}
 
+          _ ->
+            {:noreply, socket}
+        end
       _ ->
-        {:noreply, socket}
+        {:noreply, socket |> put_flash(
+          :error,
+          "You should enable tracking for at least one character to delete connection."
+        )}
     end
+
   end
 
   @impl true
@@ -1061,6 +1158,7 @@ defmodule WandererAppWeb.MapLive do
   def handle_event("toggle_track", %{"character-id" => character_id}, socket) do
     map = socket.assigns.map
     character_settings = socket.assigns.character_settings
+    current_user = socket.assigns.current_user
 
     socket =
       case character_settings |> Enum.find(&(&1.character_id == character_id)) do
@@ -1139,6 +1237,8 @@ defmodule WandererAppWeb.MapLive do
 
     {:noreply,
      socket
+     |> assign(user_characters: user_character_eve_ids)
+     |> assign(has_tracked_characters?: _has_tracked_characters?(user_character_eve_ids))
      |> assign(character_settings: character_settings)
      |> assign_async(:characters, fn ->
        {:ok, %{characters: characters}}
@@ -1275,6 +1375,15 @@ defmodule WandererAppWeb.MapLive do
               []
           end
 
+          events =
+            case map_characters |> Enum.empty?() do
+              true ->
+                events ++ [:empty_tracked_characters]
+
+              _ ->
+                events
+            end
+
         {:ok, characters_limit} = map_id |> WandererApp.Map.get_characters_limit()
 
         {:ok, present_character_ids} =
@@ -1509,6 +1618,9 @@ defmodule WandererAppWeb.MapLive do
     {:ok, %{maps: maps |> Enum.sort_by(& &1.name, :asc) |> Enum.map(&map_map/1)}}
   end
 
+  defp _has_tracked_characters?([]), do: false
+  defp _has_tracked_characters?(_user_characters), do: true
+
   defp _update_system_positions(_map_id, []), do: :ok
 
   defp _update_system_positions(map_id, [position | rest]) do

mix.exs

@@ -2,7 +2,7 @@ defmodule WandererApp.MixProject do
   use Mix.Project
 
   @source_url "https://github.com/wanderer-industries/wanderer"
-  @version "1.0.9"
+  @version "1.0.14"
 
   def project do
     [