diff --git a/CODE_GUIDELINES.md b/CODE_GUIDELINES.md index 439eee8..70e1596 100644 --- a/CODE_GUIDELINES.md +++ b/CODE_GUIDELINES.md @@ -1264,8 +1264,6 @@ end ### 3.12 Internationalization: Gettext -**German (de):** Use informal address (“duzen”). All user-facing German text should address the user as “du” (e.g. “Bitte versuche es erneut”, “Deine Einstellungen”), not “Sie”. - **Define Translations:** ```elixir diff --git a/lib/mv_web/live/group_live/show.ex b/lib/mv_web/live/group_live/show.ex index 0c7e93e..0251fb6 100644 --- a/lib/mv_web/live/group_live/show.ex +++ b/lib/mv_web/live/group_live/show.ex @@ -17,12 +17,10 @@ defmodule MvWeb.GroupLive.Show do require Logger - import Ash.Expr import MvWeb.LiveHelpers, only: [current_actor: 1] import MvWeb.Authorization alias Mv.Membership - alias MvWeb.Helpers.MemberHelpers, as: MemberHelpers @impl true def mount(_params, _session, socket) do @@ -31,7 +29,6 @@ defmodule MvWeb.GroupLive.Show do |> assign(:show_add_member_input, false) |> assign(:member_search_query, "") |> assign(:available_members, []) - |> assign(:add_member_candidates, []) |> assign(:selected_member_ids, []) |> assign(:selected_members, []) |> assign(:show_member_dropdown, false) @@ -97,21 +94,13 @@ defmodule MvWeb.GroupLive.Show do
- <%= if can?(@current_user, :update, @group) do %> - <.button - variant="primary" - navigate={~p"/groups/#{@group.slug}/edit"} - data-testid="group-show-edit-btn" - > + <%= if can?(@current_user, :update, Mv.Membership.Group) do %> + <.button variant="primary" navigate={~p"/groups/#{@group.slug}/edit"}> {gettext("Edit")} <% end %> - <%= if can?(@current_user, :destroy, @group) do %> - <.button - class="btn-error" - phx-click="open_delete_modal" - data-testid="group-show-delete-btn" - > + <%= if can?(@current_user, :destroy, Mv.Membership.Group) do %> + <.button class="btn-error" phx-click="open_delete_modal"> {gettext("Delete")} <% end %> @@ -134,7 +123,7 @@ defmodule MvWeb.GroupLive.Show do

{gettext("Members")}

-

+

{ngettext( "Total: %{count} member", "Total: %{count} members", @@ -143,7 +132,7 @@ defmodule MvWeb.GroupLive.Show do )}

- <%= if can?(@current_user, :update, @group) do %> + <%= if can?(@current_user, :update, Mv.Membership.Group) do %>
<%= if assigns[:show_add_member_input] do %>
@@ -171,7 +160,6 @@ defmodule MvWeb.GroupLive.Show do @@ -268,17 +255,15 @@ defmodule MvWeb.GroupLive.Show do <% end %> <%= if Enum.empty?(@group.members || []) do %> -

- {gettext("No members in this group")} -

+

{gettext("No members in this group")}

<% else %> -
+
- <%= if can?(@current_user, :update, @group) do %> + <%= if can?(@current_user, :update, Mv.Membership.Group) do %> <% end %> @@ -306,14 +291,13 @@ defmodule MvWeb.GroupLive.Show do <% end %> - <%= if can?(@current_user, :update, @group) do %> + <%= if can?(@current_user, :update, Mv.Membership.Group) do %>
{gettext("Name")} {gettext("Email")}{gettext("Actions")}