From 5530e152dcc66307555acbb8f4fbb19e51fc9a98 Mon Sep 17 00:00:00 2001 From: ruslandoga <67764432+ruslandoga@users.noreply.github.com> Date: Sat, 21 Dec 2024 14:15:04 +0700 Subject: [PATCH 1/3] try Elixir v1.18, Erlang 27.2 --- .tool-versions | 4 +- Dockerfile | 2 +- lib/plausible/auth/user_admin.ex | 27 ++++---- lib/plausible/crm_extensions.ex | 4 +- lib/plausible/goal/schema.ex | 2 +- lib/plausible_release.ex | 6 +- .../controllers/api/external_controller.ex | 4 +- .../live/components/verification.ex | 4 +- lib/plausible_web/live/register_form.ex | 6 +- lib/plausible_web/live/sites.ex | 66 +++++++++---------- .../plugs/authorize_public_api.ex | 38 +++++------ .../plugs/authorize_site_access.ex | 25 +++---- .../plugs/maybe_disable_registration.ex | 29 ++++---- lib/plausible_web/router.ex | 25 ++++--- .../email/site_setup_help_email.html.heex | 8 ++- .../email/site_setup_success_email.html.heex | 10 +-- .../templates/layout/_header.html.heex | 48 +++++++------- .../templates/site/settings_people.html.heex | 16 +++-- lib/plausible_web/views/layout_view.ex | 9 ++- mix.lock | 18 ++--- test/plausible/exports_test.exs | 10 ++- test/plausible/imported/csv_importer_test.exs | 4 +- .../api/stats_controller/imported_test.exs | 2 +- 23 files changed, 197 insertions(+), 170 deletions(-) diff --git a/.tool-versions b/.tool-versions index 8d70cf79e8bd..276dee0388d9 100644 --- a/.tool-versions +++ b/.tool-versions @@ -1,3 +1,3 @@ -erlang 27.0 -elixir 1.17.1-otp-27 +erlang 27.2 +elixir 1.18.0-otp-27 nodejs 21.7.3 diff --git a/Dockerfile b/Dockerfile index ec88934621cf..041e2792c1aa 100644 --- a/Dockerfile +++ b/Dockerfile @@ -2,7 +2,7 @@ # platform specific, it makes sense to build it in the docker #### Builder -FROM hexpm/elixir:1.17.1-erlang-27.0-alpine-3.20.1 AS buildcontainer +FROM hexpm/elixir:1.18.0-erlang-27.2-alpine-3.20.1 AS buildcontainer ARG MIX_ENV=ce diff --git a/lib/plausible/auth/user_admin.ex b/lib/plausible/auth/user_admin.ex index 4e24f343ebf2..7c72ae2085fe 100644 --- a/lib/plausible/auth/user_admin.ex +++ b/lib/plausible/auth/user_admin.ex @@ -171,22 +171,21 @@ defmodule Plausible.Auth.UserAdmin do defp subscription_status(user) do team = user.my_team - cond do - team && team.subscription -> - status_str = - PlausibleWeb.SettingsView.present_subscription_status(team.subscription.status) - - if team.subscription.paddle_subscription_id do - {:safe, ~s(#{status_str})} - else - status_str - end - - Plausible.Teams.on_trial?(team) -> + if team && team.subscription do + status_str = + PlausibleWeb.SettingsView.present_subscription_status(team.subscription.status) + + if team.subscription.paddle_subscription_id do + {:safe, ~s(#{status_str})} + else + status_str + end + else + if Plausible.Teams.on_trial?(team) do "On trial" - - true -> + else "Trial expired" + end end end diff --git a/lib/plausible/crm_extensions.ex b/lib/plausible/crm_extensions.ex index 054b6106f003..be9e49f4f669 100644 --- a/lib/plausible/crm_extensions.ex +++ b/lib/plausible/crm_extensions.ex @@ -48,7 +48,9 @@ defmodule Plausible.CrmExtensions do ] end - def javascripts(%{assigns: %{context: "billing", resource: "enterprise_plan", changeset: %{}}}) do + def javascripts(%{ + assigns: %{context: "billing", resource: "enterprise_plan", changeset: %{}} + }) do [ Phoenix.HTML.raw("""