diff --git a/elixir/apps/web/lib/web/live/actors/groups.ex b/elixir/apps/web/lib/web/live/actors/groups.ex index 6e25929af..a1306275d 100644 --- a/elixir/apps/web/lib/web/live/actors/groups.ex +++ b/elixir/apps/web/lib/web/live/actors/groups.ex @@ -128,7 +128,10 @@ defmodule Web.Actors.EditGroups do -
+ Note: Users will always belong to the Everyone group. +
<.button class="m-4" data-confirm={confirm_message(@added, @removed)} phx-click="submit"> Save @@ -222,7 +225,12 @@ defmodule Web.Actors.EditGroups do remove = if removed_names != [], do: "remove #{Enum.join(removed_names, ", ")}" change = [add, remove] |> Enum.reject(&is_nil/1) |> Enum.join(" and ") - "Are you sure you want to #{change}?" + if change == "" do + # Don't show confirmation message if no changes were made + nil + else + "Are you sure you want to #{change}?" + end end defp remove_non_editable_memberships(memberships, editable_groups) do diff --git a/elixir/apps/web/lib/web/live/groups/edit_actors.ex b/elixir/apps/web/lib/web/live/groups/edit_actors.ex index 3bbeec7e0..309568bc4 100644 --- a/elixir/apps/web/lib/web/live/groups/edit_actors.ex +++ b/elixir/apps/web/lib/web/live/groups/edit_actors.ex @@ -232,6 +232,11 @@ defmodule Web.Groups.EditActors do remove = if removed_names != [], do: "remove #{Enum.join(removed_names, ", ")}" change = [add, remove] |> Enum.reject(&is_nil/1) |> Enum.join(" and ") - "Are you sure you want to #{change}?" + if change == "" do + # Don't show confirmation message if no changes were made + nil + else + "Are you sure you want to #{change}?" + end end end