From 91a1141441d450f7caab3ba101f7fff7deab6469 Mon Sep 17 00:00:00 2001 From: Claudio Ortolina Date: Fri, 7 Mar 2025 15:10:22 +0000 Subject: [PATCH] Rename Import -> Add, and avoid surfacing MusicBrainz where unneeded --- README.md | 8 +-- .../components/barcode_scanner_component.ex | 4 +- .../components/import_component.ex | 4 +- .../live/collection_live/index.ex | 2 +- .../live/collection_live/index.html.heex | 2 +- .../live/wishlist_live/index.ex | 2 +- .../live/wishlist_live/index.html.heex | 4 +- priv/gettext/default.pot | 68 +++++++++---------- .../live/collection_live/index_test.exs | 10 +-- 9 files changed, 50 insertions(+), 54 deletions(-) diff --git a/README.md b/README.md index c617905a..1049abdc 100644 --- a/README.md +++ b/README.md @@ -8,7 +8,7 @@ - [Collection](#collection) - [Edit a record in the collection](#edit-a-record-in-the-collection) - [View a record's details in the collection](#view-a-records-details-in-the-collection) - - [Importing a record in the wishlist](#importing-a-record-in-the-wishlist) + - [Adding a record in the wishlist](#importing-a-record-in-the-wishlist) - [Setup](#setup) - [Environment configuration](#environment-configuration) - [Running the application](#running-the-application) @@ -19,7 +19,7 @@ ## Features -- Import records from MusicBrainz, with optional override of specific pieces of data +- Add records from MusicBrainz, with optional override of specific pieces of data - Manage a collection and a wishlist of records, with ways to quickly search and filter based on records' metadata - Integration with Last.fm (display latest scrobbles, and where possible @@ -46,9 +46,9 @@ ![View a record's details in the collection](.github/screenshots/collection-record-details.png) -### Importing a record in the wishlist +### Adding a record in the wishlist -![Importing a record in the wishlist](.github/screenshots/wishlist-import-record.png) +![Adding a record in the wishlist](.github/screenshots/wishlist-import-record.png) ## Setup diff --git a/lib/music_library_web/components/barcode_scanner_component.ex b/lib/music_library_web/components/barcode_scanner_component.ex index ec8b4b1b..ab5cb28d 100644 --- a/lib/music_library_web/components/barcode_scanner_component.ex +++ b/lib/music_library_web/components/barcode_scanner_component.ex @@ -50,10 +50,10 @@ defmodule MusicLibraryWeb.BarcodeScannerComponent do
<.button disabled={length(@scan_results) == 0} - phx-disable-with={gettext("Importing...")} + phx-disable-with={gettext("Adding...")} phx-click={JS.push("import_releases", target: "#barcode-scanner")} > - {gettext("Import releases")} + {gettext("Add releases")}
diff --git a/lib/music_library_web/components/import_component.ex b/lib/music_library_web/components/import_component.ex index fefb1521..32aae00c 100644 --- a/lib/music_library_web/components/import_component.ex +++ b/lib/music_library_web/components/import_component.ex @@ -24,8 +24,8 @@ defmodule MusicLibraryWeb.ImportComponent do name={:mb_query} field={@form[:mb_query]} type="text" - label={gettext("Search for a record on MusicBrainz")} - prompt={gettext("Search for records")} + label={gettext("Search for a record")} + prompt={gettext("Search for a record")} phx-debounce="500" autocorrect="off" autocapitalize="none" diff --git a/lib/music_library_web/live/collection_live/index.ex b/lib/music_library_web/live/collection_live/index.ex index b24bb192..e5900344 100644 --- a/lib/music_library_web/live/collection_live/index.ex +++ b/lib/music_library_web/live/collection_live/index.ex @@ -41,7 +41,7 @@ defmodule MusicLibraryWeb.CollectionLive.Index do end socket - |> assign(:page_title, gettext("Import from MusicBrainz · Collection")) + |> assign(:page_title, gettext("Add new Record · Collection")) |> assign(:record, nil) end diff --git a/lib/music_library_web/live/collection_live/index.html.heex b/lib/music_library_web/live/collection_live/index.html.heex index 8a2ecb77..e3c3b8b5 100644 --- a/lib/music_library_web/live/collection_live/index.html.heex +++ b/lib/music_library_web/live/collection_live/index.html.heex @@ -15,7 +15,7 @@ patch={~p"/collection/import"} > <.icon name="hero-plus" class="h-4 w-4 mr-2" aria-hidden="true" data-slot="icon" /> - {gettext("Import")} + {gettext("Add")} <.link class={[ diff --git a/lib/music_library_web/live/wishlist_live/index.ex b/lib/music_library_web/live/wishlist_live/index.ex index 7ad829eb..e88a771a 100644 --- a/lib/music_library_web/live/wishlist_live/index.ex +++ b/lib/music_library_web/live/wishlist_live/index.ex @@ -46,7 +46,7 @@ defmodule MusicLibraryWeb.WishlistLive.Index do end socket - |> assign(:page_title, gettext("Import from MusicBrainz · Wishlist")) + |> assign(:page_title, gettext("Add new Record · Wishlist")) |> assign(:record, nil) end diff --git a/lib/music_library_web/live/wishlist_live/index.html.heex b/lib/music_library_web/live/wishlist_live/index.html.heex index 9a899cf4..543c0a44 100644 --- a/lib/music_library_web/live/wishlist_live/index.html.heex +++ b/lib/music_library_web/live/wishlist_live/index.html.heex @@ -14,7 +14,7 @@ patch={~p"/wishlist/import"} > <.icon name="hero-plus" class="h-4 w-4 mr-2" aria-hidden="true" data-slot="icon" /> - {gettext("Import")} + {gettext("Add")} @@ -65,7 +65,7 @@ record={@record} patch={back_path(@record_list_params)} initial_query="" - icon_name="hero-star" + icon_name="hero-plus" /> diff --git a/priv/gettext/default.pot b/priv/gettext/default.pot index d2325b69..f59418db 100644 --- a/priv/gettext/default.pot +++ b/priv/gettext/default.pot @@ -95,12 +95,6 @@ msgstr "" msgid "Hang in there while we get back on track" msgstr "" -#: lib/music_library_web/live/collection_live/index.html.heex:18 -#: lib/music_library_web/live/wishlist_live/index.html.heex:17 -#, elixir-autogen, elixir-format -msgid "Import" -msgstr "" - #: lib/music_library_web/live/collection_live/show.html.heex:231 #: lib/music_library_web/live/wishlist_live/show.html.heex:243 #, elixir-autogen, elixir-format @@ -213,16 +207,6 @@ msgstr "" msgid "Search" msgstr "" -#: lib/music_library_web/components/import_component.ex:27 -#, elixir-autogen, elixir-format -msgid "Search for a record on MusicBrainz" -msgstr "" - -#: lib/music_library_web/components/import_component.ex:28 -#, elixir-autogen, elixir-format -msgid "Search for records" -msgstr "" - #: lib/music_library_web/components/record_components.ex:140 #: lib/music_library_web/live/collection_live/show.ex:140 #: lib/music_library_web/live/wishlist_live/show.ex:143 @@ -400,16 +384,6 @@ msgstr "" msgid "Details" msgstr "" -#: lib/music_library_web/live/collection_live/index.ex:44 -#, elixir-autogen, elixir-format -msgid "Import from MusicBrainz · Collection" -msgstr "" - -#: lib/music_library_web/live/wishlist_live/index.ex:49 -#, elixir-autogen, elixir-format -msgid "Import from MusicBrainz · Wishlist" -msgstr "" - #: lib/music_library_web/components/layouts/root.html.heex:59 #, elixir-autogen, elixir-format msgid "Made by" @@ -626,21 +600,11 @@ msgstr "" msgid "Open camera" msgstr "" -#: lib/music_library_web/components/barcode_scanner_component.ex:56 -#, elixir-autogen, elixir-format -msgid "Import releases" -msgstr "" - #: lib/music_library_web/components/barcode_scanner_component.ex:187 #, elixir-autogen, elixir-format msgid "Failed to search release for barcode %{number}" msgstr "" -#: lib/music_library_web/components/barcode_scanner_component.ex:53 -#, elixir-autogen, elixir-format -msgid "Importing..." -msgstr "" - #: lib/music_library_web/components/barcode_scanner_component.ex:200 #, elixir-autogen, elixir-format msgid "Records imported successfully" @@ -695,3 +659,35 @@ msgstr "" #, elixir-autogen, elixir-format msgid "Some records could not be imported: %{summary}" msgstr "" + +#: lib/music_library_web/live/collection_live/index.html.heex:18 +#: lib/music_library_web/live/wishlist_live/index.html.heex:17 +#, elixir-autogen, elixir-format +msgid "Add" +msgstr "" + +#: lib/music_library_web/live/collection_live/index.ex:44 +#, elixir-autogen, elixir-format +msgid "Add new Record · Collection" +msgstr "" + +#: lib/music_library_web/live/wishlist_live/index.ex:49 +#, elixir-autogen, elixir-format +msgid "Add new Record · Wishlist" +msgstr "" + +#: lib/music_library_web/components/barcode_scanner_component.ex:56 +#, elixir-autogen, elixir-format +msgid "Add releases" +msgstr "" + +#: lib/music_library_web/components/barcode_scanner_component.ex:53 +#, elixir-autogen, elixir-format +msgid "Adding..." +msgstr "" + +#: lib/music_library_web/components/import_component.ex:27 +#: lib/music_library_web/components/import_component.ex:28 +#, elixir-autogen, elixir-format +msgid "Search for a record" +msgstr "" diff --git a/test/music_library_web/live/collection_live/index_test.exs b/test/music_library_web/live/collection_live/index_test.exs index 9758cbc0..98c02f05 100644 --- a/test/music_library_web/live/collection_live/index_test.exs +++ b/test/music_library_web/live/collection_live/index_test.exs @@ -229,12 +229,12 @@ defmodule MusicLibraryWeb.CollectionLive.IndexTest do end end - describe "Importing a new record" do + describe "Adding a new record" do test "it shows the import modal", %{conn: conn} do conn |> visit(~p"/collection") - |> click_link("Import") - |> assert_has("label", text: "Search for a record on MusicBrainz") + |> click_link("Add") + |> assert_has("label", text: "Search for a record") |> assert_has("div", text: "No results") |> assert_path(~p"/collection/import") end @@ -269,7 +269,7 @@ defmodule MusicLibraryWeb.CollectionLive.IndexTest do session = conn |> visit(~p"/collection/import") - |> fill_in("Search for a record on MusicBrainz", with: "Marillion Marbles") + |> fill_in("Search for a record", with: "Marillion Marbles") for release_group_search_result <- release_group_search_results do result = ReleaseGroupSearchResult.from_api_response(release_group_search_result) @@ -386,7 +386,7 @@ defmodule MusicLibraryWeb.CollectionLive.IndexTest do |> trigger_hook("#barcode-scanner", "barcode_scanned", %{"number" => barcode}) |> assert_has("h2", text: "Marbles") |> assert_has("span", text: "New") - |> click_button("Import releases") + |> click_button("Add releases") [record] = MusicLibrary.Repo.all(MusicLibrary.Records.Record)