Added translation using Weblate (German)

improve tagging logic
fix user invite page
2022-12-20 03:35:57 +00:00 · 2022-12-19 22:34:00 -05:00 · 2022-12-19 21:14:14 -05:00 · 2022-12-19 21:09:57 -05:00 · 2022-12-19 21:09:50 -05:00
29 changed files with 207 additions and 148 deletions
--- a/changelog.md
+++ b/changelog.md
@ -1,3 +1,9 @@
+# v0.1.4
+- fix docker-compose
+- fix newlines in notes/context/step contents
+- fix user invite page
+- improve tagging logic
+
 # v0.1.3
 - backlink to other notes in notes
 - search tags on click
--- a/de.tbx
+++ b/de.tbx
@ -0,0 +1,10 @@
+<?xml version="1.0"?>
+<!DOCTYPE martif PUBLIC "ISO 12200:1999A//DTD MARTIF core (DXFcdV04)//EN" "TBXcdv04.dtd">
+<martif type="TBX">
+<martifHeader>
+<fileDesc>
+<sourceDesc><p>Translate Toolkit</p></sourceDesc>
+</fileDesc>
+</martifHeader>
+<text><body></body></text>
+</martif>
--- a/docker-compose.yml
+++ b/docker-compose.yml
@ -2,8 +2,7 @@ version: '3'

 services:
  memex:
-    build:
-      context: .
+    image: shibaobun/memex
    container_name: memex
    restart: always
    environment:
--- a/lib/memex/contexts.ex
+++ b/lib/memex/contexts.ex
@ -4,7 +4,6 @@ defmodule Memex.Contexts do
  """

  import Ecto.Query, warn: false
-  alias Ecto.Changeset
  alias Memex.{Accounts.User, Contexts.Context, Repo}

  @doc """
@ -229,18 +228,4 @@ defmodule Memex.Contexts do
  def change_context(%Context{} = context, attrs \\ %{}, user) do
    context |> Context.update_changeset(attrs, user)
  end
-
-  @doc """
-  Gets a canonical string representation of the `:tags` field for a Note
-  """
-  @spec get_tags_string(Context.t() | Context.changeset() | [String.t()] | nil) :: String.t()
-  def get_tags_string(nil), do: ""
-  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
-  def get_tags_string(%Context{tags: tags}), do: tags |> get_tags_string()
-
-  def get_tags_string(%Changeset{} = changeset) do
-    changeset
-    |> Changeset.get_field(:tags)
-    |> get_tags_string()
-  end
 end
--- a/lib/memex/contexts/context.ex
+++ b/lib/memex/contexts/context.ex
@ -27,7 +27,7 @@ defmodule Memex.Contexts.Context do
          slug: slug(),
          content: String.t(),
          tags: [String.t()] | nil,
-          tags_string: String.t(),
+          tags_string: String.t() | nil,
          visibility: :public | :private | :unlisted,
          user: User.t() | Ecto.Association.NotLoaded.t(),
          user_id: User.id(),
@ -66,16 +66,38 @@ defmodule Memex.Contexts.Context do
    |> unsafe_validate_unique(:slug, Repo)
  end

-  defp cast_tags_string(changeset, %{"tags_string" => tags_string})
-       when tags_string |> is_binary() do
-    tags =
+  defp cast_tags_string(changeset, attrs) do
+    changeset
+    |> put_change(:tags_string, changeset |> get_field(:tags) |> get_tags_string())
+    |> cast(attrs, [:tags_string])
+    |> validate_format(:tags_string, ~r/^[\p{L}\p{N}\-\,]+$/,
+      message:
+        dgettext(
+          "errors",
+          "invalid format: only numbers, letters and hyphen are accepted. tags must be comma-delimited"
+        )
+    )
+    |> cast_tags()
+  end
+
+  defp cast_tags(%{valid?: false} = changeset), do: changeset
+
+  defp cast_tags(%{valid?: true} = changeset) do
+    tags = changeset |> get_field(:tags_string) |> process_tags()
+    changeset |> put_change(:tags, tags)
+  end
+
+  defp process_tags(tags_string) when tags_string |> is_binary() do
    tags_string
    |> String.split(",", trim: true)
    |> Enum.map(fn str -> str |> String.trim() end)
+    |> Enum.reject(fn str -> str |> is_nil() end)
    |> Enum.sort()
-
-    changeset |> change(tags: tags)
  end

-  defp cast_tags_string(changeset, _attrs), do: changeset
+  defp process_tags(_other_tags_string), do: []
+
+  @spec get_tags_string([String.t()] | nil) :: String.t()
+  def get_tags_string(nil), do: ""
+  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
 end
--- a/lib/memex/notes.ex
+++ b/lib/memex/notes.ex
@ -4,7 +4,6 @@ defmodule Memex.Notes do
  """

  import Ecto.Query, warn: false
-  alias Ecto.Changeset
  alias Memex.{Accounts.User, Notes.Note, Repo}

  @doc """
@ -229,18 +228,4 @@ defmodule Memex.Notes do
  def change_note(%Note{} = note, attrs \\ %{}, user) do
    note |> Note.update_changeset(attrs, user)
  end
-
-  @doc """
-  Gets a canonical string representation of the `:tags` field for a Note
-  """
-  @spec get_tags_string(Note.t() | Note.changeset() | [String.t()] | nil) :: String.t()
-  def get_tags_string(nil), do: ""
-  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
-  def get_tags_string(%Note{tags: tags}), do: tags |> get_tags_string()
-
-  def get_tags_string(%Changeset{} = changeset) do
-    changeset
-    |> Changeset.get_field(:tags)
-    |> get_tags_string()
-  end
 end
--- a/lib/memex/notes/note.ex
+++ b/lib/memex/notes/note.ex
@ -26,7 +26,7 @@ defmodule Memex.Notes.Note do
          slug: slug(),
          content: String.t(),
          tags: [String.t()] | nil,
-          tags_string: String.t(),
+          tags_string: String.t() | nil,
          visibility: :public | :private | :unlisted,
          user: User.t() | Ecto.Association.NotLoaded.t(),
          user_id: User.id(),
@ -65,16 +65,38 @@ defmodule Memex.Notes.Note do
    |> unsafe_validate_unique(:slug, Repo)
  end

-  defp cast_tags_string(changeset, %{"tags_string" => tags_string})
-       when tags_string |> is_binary() do
-    tags =
+  defp cast_tags_string(changeset, attrs) do
+    changeset
+    |> put_change(:tags_string, changeset |> get_field(:tags) |> get_tags_string())
+    |> cast(attrs, [:tags_string])
+    |> validate_format(:tags_string, ~r/^[\p{L}\p{N}\-\,]+$/,
+      message:
+        dgettext(
+          "errors",
+          "invalid format: only numbers, letters and hyphen are accepted. tags must be comma-delimited"
+        )
+    )
+    |> cast_tags()
+  end
+
+  defp cast_tags(%{valid?: false} = changeset), do: changeset
+
+  defp cast_tags(%{valid?: true} = changeset) do
+    tags = changeset |> get_field(:tags_string) |> process_tags()
+    changeset |> put_change(:tags, tags)
+  end
+
+  defp process_tags(tags_string) when tags_string |> is_binary() do
    tags_string
    |> String.split(",", trim: true)
    |> Enum.map(fn str -> str |> String.trim() end)
+    |> Enum.reject(fn str -> str |> is_nil() end)
    |> Enum.sort()
-
-    changeset |> change(tags: tags)
  end

-  defp cast_tags_string(changeset, _attrs), do: changeset
+  defp process_tags(_other_tags_string), do: []
+
+  @spec get_tags_string([String.t()] | nil) :: String.t()
+  def get_tags_string(nil), do: ""
+  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
 end
--- a/lib/memex/pipelines.ex
+++ b/lib/memex/pipelines.ex
@ -4,7 +4,6 @@ defmodule Memex.Pipelines do
  """

  import Ecto.Query, warn: false
-  alias Ecto.Changeset
  alias Memex.{Accounts.User, Pipelines.Pipeline, Repo}

  @doc """
@ -231,18 +230,4 @@ defmodule Memex.Pipelines do
  def change_pipeline(%Pipeline{} = pipeline, attrs \\ %{}, user) do
    pipeline |> Pipeline.update_changeset(attrs, user)
  end
-
-  @doc """
-  Gets a canonical string representation of the `:tags` field for a Pipeline
-  """
-  @spec get_tags_string(Pipeline.t() | Pipeline.changeset() | [String.t()] | nil) :: String.t()
-  def get_tags_string(nil), do: ""
-  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
-  def get_tags_string(%Pipeline{tags: tags}), do: tags |> get_tags_string()
-
-  def get_tags_string(%Changeset{} = changeset) do
-    changeset
-    |> Changeset.get_field(:tags)
-    |> get_tags_string()
-  end
 end
--- a/lib/memex/pipelines/pipeline.ex
+++ b/lib/memex/pipelines/pipeline.ex
@ -28,7 +28,7 @@ defmodule Memex.Pipelines.Pipeline do
          slug: slug(),
          description: String.t(),
          tags: [String.t()] | nil,
-          tags_string: String.t(),
+          tags_string: String.t() | nil,
          visibility: :public | :private | :unlisted,
          user: User.t() | Ecto.Association.NotLoaded.t(),
          user_id: User.id(),
@ -67,16 +67,38 @@ defmodule Memex.Pipelines.Pipeline do
    |> unsafe_validate_unique(:slug, Repo)
  end

-  defp cast_tags_string(changeset, %{"tags_string" => tags_string})
-       when tags_string |> is_binary() do
-    tags =
+  defp cast_tags_string(changeset, attrs) do
+    changeset
+    |> put_change(:tags_string, changeset |> get_field(:tags) |> get_tags_string())
+    |> cast(attrs, [:tags_string])
+    |> validate_format(:tags_string, ~r/^[\p{L}\p{N}\-\,]+$/,
+      message:
+        dgettext(
+          "errors",
+          "invalid format: only numbers, letters and hyphen are accepted. tags must be comma-delimited"
+        )
+    )
+    |> cast_tags()
+  end
+
+  defp cast_tags(%{valid?: false} = changeset), do: changeset
+
+  defp cast_tags(%{valid?: true} = changeset) do
+    tags = changeset |> get_field(:tags_string) |> process_tags()
+    changeset |> put_change(:tags, tags)
+  end
+
+  defp process_tags(tags_string) when tags_string |> is_binary() do
    tags_string
    |> String.split(",", trim: true)
    |> Enum.map(fn str -> str |> String.trim() end)
+    |> Enum.reject(fn str -> str |> is_nil() end)
    |> Enum.sort()
-
-    changeset |> change(tags: tags)
  end

-  defp cast_tags_string(changeset, _attrs), do: changeset
+  defp process_tags(_other_tags_string), do: []
+
+  @spec get_tags_string([String.t()] | nil) :: String.t()
+  def get_tags_string(nil), do: ""
+  def get_tags_string(tags) when tags |> is_list(), do: tags |> Enum.join(",")
 end
--- a/lib/memex_web/components/context_content.ex
+++ b/lib/memex_web/components/context_content.ex
@ -39,6 +39,7 @@ defmodule MemexWeb.Components.ContextContent do
        "</p>#{link}<p class=\"inline\">"
      end
    )
+    |> String.replace("\n", "<br>")
    |> HTML.raw()
  end
 end
--- a/lib/memex_web/components/note_content.ex
+++ b/lib/memex_web/components/note_content.ex
@ -39,6 +39,7 @@ defmodule MemexWeb.Components.NoteContent do
        "</p>#{link}<p class=\"inline\">"
      end
    )
+    |> String.replace("\n", "<br>")
    |> HTML.raw()
  end
 end
--- a/lib/memex_web/components/step_content.ex
+++ b/lib/memex_web/components/step_content.ex
@ -39,6 +39,7 @@ defmodule MemexWeb.Components.StepContent do
        "</p>#{link}<p class=\"inline\">"
      end
    )
+    |> String.replace("\n", "<br>")
    |> HTML.raw()
  end
 end
--- a/lib/memex_web/components/user_card.ex
+++ b/lib/memex_web/components/user_card.ex
@ -22,15 +22,15 @@ defmodule MemexWeb.Components.UserCard do
          <%= if @user.confirmed_at |> is_nil() do %>
            <%= gettext("email unconfirmed") %>
          <% else %>
-            <%= gettext(
-              "user was confirmed at %{relative_datetime}",
-              relative_datetime: @user.confirmed_at |> display_datetime()
-            ) %>
+            <p>
+              <%= gettext("user confirmed on") %>
+              <%= @user.confirmed_at |> display_datetime() %>
+            </p>
          <% end %>
        </p>

        <p>
-          <%= gettext("User registered on") %>
+          <%= gettext("user registered on") %>
          <%= @user.inserted_at |> display_datetime() %>
        </p>
      </h3>
--- a/lib/memex_web/live/context_live/form_component.html.heex
+++ b/lib/memex_web/live/context_live/form_component.html.heex
@ -27,9 +27,7 @@
    <%= text_input(f, :tags_string,
      id: "tags-input",
      class: "input input-primary",
-      placeholder: gettext("tag1,tag2"),
-      phx_update: "ignore",
-      value: Contexts.get_tags_string(@changeset)
+      placeholder: gettext("tag1,tag2")
    ) %>
    <%= error_tag(f, :tags_string) %>

--- a/lib/memex_web/live/note_live/form_component.html.heex
+++ b/lib/memex_web/live/note_live/form_component.html.heex
@ -27,9 +27,7 @@
    <%= text_input(f, :tags_string,
      id: "tags-input",
      class: "input input-primary",
-      placeholder: gettext("tag1,tag2"),
-      phx_update: "ignore",
-      value: Notes.get_tags_string(@changeset)
+      placeholder: gettext("tag1,tag2")
    ) %>
    <%= error_tag(f, :tags_string) %>

--- a/lib/memex_web/live/pipeline_live/form_component.html.heex
+++ b/lib/memex_web/live/pipeline_live/form_component.html.heex
@ -27,9 +27,7 @@
    <%= text_input(f, :tags_string,
      id: "tags-input",
      class: "input input-primary",
-      placeholder: gettext("tag1,tag2"),
-      phx_update: "ignore",
-      value: Pipelines.get_tags_string(@changeset)
+      placeholder: gettext("tag1,tag2")
    ) %>
    <%= error_tag(f, :tags_string) %>

--- a/mix.exs
+++ b/mix.exs
@ -4,7 +4,7 @@ defmodule Memex.MixProject do
  def project do
    [
      app: :memex,
-      version: "0.1.3",
+      version: "0.1.4",
      elixir: "~> 1.14",
      elixirc_paths: elixirc_paths(Mix.env()),
      compilers: Mix.compilers(),
--- a/priv/gettext/actions.pot
+++ b/priv/gettext/actions.pot
@ -124,9 +124,9 @@ msgstr ""
 msgid "register"
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:42
-#: lib/memex_web/live/note_live/form_component.html.heex:42
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:42
+#: lib/memex_web/live/context_live/form_component.html.heex:40
+#: lib/memex_web/live/note_live/form_component.html.heex:40
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:40
 #: lib/memex_web/live/step_live/form_component.html.heex:28
 #, elixir-autogen, elixir-format
 msgid "save"
--- a/priv/gettext/de/LC_MESSAGES/actions.po
+++ b/priv/gettext/de/LC_MESSAGES/actions.po
@ -125,9 +125,9 @@ msgstr ""
 msgid "register"
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:42
-#: lib/memex_web/live/note_live/form_component.html.heex:42
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:42
+#: lib/memex_web/live/context_live/form_component.html.heex:40
+#: lib/memex_web/live/note_live/form_component.html.heex:40
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:40
 #: lib/memex_web/live/step_live/form_component.html.heex:28
 #, elixir-autogen, elixir-format
 msgid "save"
--- a/priv/gettext/de/LC_MESSAGES/default.po
+++ b/priv/gettext/de/LC_MESSAGES/default.po
@ -66,11 +66,6 @@ msgstr ""
 msgid "Settings"
 msgstr ""

-#: lib/memex_web/components/user_card.ex:33
-#, elixir-autogen, elixir-format
-msgid "User registered on"
-msgstr ""
-
 #: lib/memex_web/components/invite_card.ex:19
 #, elixir-autogen, elixir-format
 msgid "Uses Left:"
@ -306,17 +301,17 @@ msgstr ""
 msgid "report bugs or request features"
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:43
-#: lib/memex_web/live/note_live/form_component.html.heex:43
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:43
+#: lib/memex_web/live/context_live/form_component.html.heex:41
+#: lib/memex_web/live/note_live/form_component.html.heex:41
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:41
 #: lib/memex_web/live/step_live/form_component.html.heex:29
 #, elixir-autogen, elixir-format
 msgid "saving..."
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:39
-#: lib/memex_web/live/note_live/form_component.html.heex:39
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:39
+#: lib/memex_web/live/context_live/form_component.html.heex:37
+#: lib/memex_web/live/note_live/form_component.html.heex:37
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:37
 #, elixir-autogen, elixir-format
 msgid "select privacy"
 msgstr ""
@ -350,11 +345,6 @@ msgstr ""
 msgid "unlimited"
 msgstr ""

-#: lib/memex_web/components/user_card.ex:25
-#, elixir-autogen, elixir-format
-msgid "user was confirmed at %{relative_datetime}"
-msgstr ""
-
 #: lib/memex_web/live/invite_live/index.html.heex:120
 #, elixir-autogen, elixir-format
 msgid "users"
@ -665,3 +655,13 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "while memEx fully supports multiple users, each memEx instance should be treated as a single cohesive and collaborative document."
 msgstr ""
+
+#: lib/memex_web/components/user_card.ex:26
+#, elixir-autogen, elixir-format
+msgid "user confirmed on"
+msgstr ""
+
+#: lib/memex_web/components/user_card.ex:33
+#, elixir-autogen, elixir-format, fuzzy
+msgid "user registered on"
+msgstr ""
--- a/priv/gettext/de/LC_MESSAGES/errors.po
+++ b/priv/gettext/de/LC_MESSAGES/errors.po
@ -131,3 +131,10 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "unauthorized"
 msgstr ""
+
+#: lib/memex/contexts/context.ex:75
+#: lib/memex/notes/note.ex:74
+#: lib/memex/pipelines/pipeline.ex:76
+#, elixir-autogen, elixir-format, fuzzy
+msgid "invalid format: only numbers, letters and hyphen are accepted. tags must be comma-delimited"
+msgstr ""
--- a/priv/gettext/default.pot
+++ b/priv/gettext/default.pot
@ -55,11 +55,6 @@ msgstr ""
 msgid "Settings"
 msgstr ""

-#: lib/memex_web/components/user_card.ex:33
-#, elixir-autogen, elixir-format
-msgid "User registered on"
-msgstr ""
-
 #: lib/memex_web/components/invite_card.ex:19
 #, elixir-autogen, elixir-format
 msgid "Uses Left:"
@ -295,17 +290,17 @@ msgstr ""
 msgid "report bugs or request features"
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:43
-#: lib/memex_web/live/note_live/form_component.html.heex:43
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:43
+#: lib/memex_web/live/context_live/form_component.html.heex:41
+#: lib/memex_web/live/note_live/form_component.html.heex:41
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:41
 #: lib/memex_web/live/step_live/form_component.html.heex:29
 #, elixir-autogen, elixir-format
 msgid "saving..."
 msgstr ""

-#: lib/memex_web/live/context_live/form_component.html.heex:39
-#: lib/memex_web/live/note_live/form_component.html.heex:39
-#: lib/memex_web/live/pipeline_live/form_component.html.heex:39
+#: lib/memex_web/live/context_live/form_component.html.heex:37
+#: lib/memex_web/live/note_live/form_component.html.heex:37
+#: lib/memex_web/live/pipeline_live/form_component.html.heex:37
 #, elixir-autogen, elixir-format
 msgid "select privacy"
 msgstr ""
@ -339,11 +334,6 @@ msgstr ""
 msgid "unlimited"
 msgstr ""

-#: lib/memex_web/components/user_card.ex:25
-#, elixir-autogen, elixir-format
-msgid "user was confirmed at %{relative_datetime}"
-msgstr ""
-
 #: lib/memex_web/live/invite_live/index.html.heex:120
 #, elixir-autogen, elixir-format
 msgid "users"
@ -654,3 +644,13 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "while memEx fully supports multiple users, each memEx instance should be treated as a single cohesive and collaborative document."
 msgstr ""
+
+#: lib/memex_web/components/user_card.ex:26
+#, elixir-autogen, elixir-format
+msgid "user confirmed on"
+msgstr ""
+
+#: lib/memex_web/components/user_card.ex:33
+#, elixir-autogen, elixir-format
+msgid "user registered on"
+msgstr ""
--- a/priv/gettext/errors.pot
+++ b/priv/gettext/errors.pot
@ -130,3 +130,10 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "unauthorized"
 msgstr ""
+
+#: lib/memex/contexts/context.ex:75
+#: lib/memex/notes/note.ex:74
+#: lib/memex/pipelines/pipeline.ex:76
+#, elixir-autogen, elixir-format
+msgid "invalid format: only numbers, letters and hyphen are accepted. tags must be comma-delimited"
+msgstr ""
--- a/test/memex_web/live/context_live_test.exs
+++ b/test/memex_web/live/context_live_test.exs
@ -18,7 +18,7 @@ defmodule MemexWeb.ContextLiveTest do
  }
  @invalid_attrs %{
    "content" => nil,
-    "tags_string" => "",
+    "tags_string" => " ",
    "slug" => nil,
    "visibility" => nil
  }
@ -114,9 +114,13 @@ defmodule MemexWeb.ContextLiveTest do

      assert_patch(show_live, Routes.context_show_path(conn, :edit, context.slug))

-      assert show_live
+      html =
+        show_live
        |> form("#context-form", context: @invalid_attrs)
-             |> render_change() =~ "can&#39;t be blank"
+        |> render_change()
+
+      assert html =~ "can&#39;t be blank"
+      assert html =~ "tags must be comma-delimited"

      {:ok, _, html} =
        show_live
--- a/test/memex_web/live/note_live_test.exs
+++ b/test/memex_web/live/note_live_test.exs
@ -19,7 +19,7 @@ defmodule MemexWeb.NoteLiveTest do
  }
  @invalid_attrs %{
    "content" => nil,
-    "tags_string" => "",
+    "tags_string" => " ",
    "slug" => nil,
    "visibility" => nil
  }
@ -54,9 +54,13 @@ defmodule MemexWeb.NoteLiveTest do

      assert_patch(index_live, Routes.note_index_path(conn, :new))

-      assert index_live
+      html =
+        index_live
        |> form("#note-form", note: @invalid_attrs)
-             |> render_change() =~ "can&#39;t be blank"
+        |> render_change()
+
+      assert html =~ "can&#39;t be blank"
+      assert html =~ "tags must be comma-delimited"

      {:ok, _, html} =
        index_live
--- a/test/memex_web/live/pipeline_live_test.exs
+++ b/test/memex_web/live/pipeline_live_test.exs
@ -17,7 +17,7 @@ defmodule MemexWeb.PipelineLiveTest do
  }
  @invalid_attrs %{
    "description" => nil,
-    "tags_string" => "",
+    "tags_string" => " ",
    "slug" => nil,
    "visibility" => nil
  }
@ -128,9 +128,13 @@ defmodule MemexWeb.PipelineLiveTest do

      assert_patch(show_live, Routes.pipeline_show_path(conn, :edit, pipeline.slug))

-      assert show_live
+      html =
+        show_live
        |> form("#pipeline-form", pipeline: @invalid_attrs)
-             |> render_change() =~ "can&#39;t be blank"
+        |> render_change()
+
+      assert html =~ "can&#39;t be blank"
+      assert html =~ "tags must be comma-delimited"

      {:ok, _, html} =
        show_live
--- a/test/support/fixtures/contexts_fixtures.ex
+++ b/test/support/fixtures/contexts_fixtures.ex
@ -22,6 +22,6 @@ defmodule Memex.ContextsFixtures do
      })
      |> Contexts.create_context(user)

-    context
+    %{context | tags_string: nil}
  end
 end
--- a/test/support/fixtures/notes_fixtures.ex
+++ b/test/support/fixtures/notes_fixtures.ex
@ -22,6 +22,6 @@ defmodule Memex.NotesFixtures do
      })
      |> Notes.create_note(user)

-    note
+    %{note | tags_string: nil}
  end
 end
--- a/test/support/fixtures/pipelines_fixtures.ex
+++ b/test/support/fixtures/pipelines_fixtures.ex
@ -22,6 +22,6 @@ defmodule Memex.PipelinesFixtures do
      })
      |> Pipelines.create_pipeline(user)

-    pipeline
+    %{pipeline | tags_string: nil}
  end
 end
Author	SHA1	Message	Date
Weblate	634891ee73	Added translation using Weblate (German) Some checks are pending continuous-integration/drone/push Build is running Details	2022-12-20 03:35:57 +00:00
shibao	571f6fffdb	improve tagging logic Some checks are pending continuous-integration/drone/push Build is running Details	2022-12-19 22:34:00 -05:00
shibao	926805ba9b	fix user invite page	2022-12-19 21:14:14 -05:00
shibao	220122dec6	add newlines to content components	2022-12-19 21:09:57 -05:00
shibao	de399b4819	fix broken docker-compose	2022-12-19 21:09:50 -05:00