Change locale translation route to better reflect the domain

virolea · Sep 12, 2024 · aa84dd8 · aa84dd8
1 parent 47030ac
commit aa84dd8
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 10 deletions.
diff --git a/...ta/locales/translation_keys_controller.rb → ...osetta/locales/translations_controller.rb b/...ta/locales/translation_keys_controller.rb → ...osetta/locales/translations_controller.rb
@@ -1,5 +1,5 @@
 module Rosetta
-  class Locales::TranslationKeysController < ApplicationController
+  class Locales::TranslationsController < ApplicationController
     before_action :set_locale
 
     def index

diff --git a/app/controllers/rosetta/translations_controller.rb b/app/controllers/rosetta/translations_controller.rb
@@ -9,7 +9,7 @@ def edit
 
     def update
       if @translation.update(translation_params)
-        redirect_to locale_translation_keys_path(@locale)
+        redirect_to locale_translations_path(@locale)
       end
     end
 

diff --git a/app/views/rosetta/locales/_locale.html.erb b/app/views/rosetta/locales/_locale.html.erb
@@ -13,7 +13,7 @@
     <% end %>
 
     <% unless locale.default_locale? %>
-      <%= link_to "Manage", locale_translation_keys_path(locale), class: "text-indigo-600 hover:text-indigo-900" %>
+      <%= link_to "Manage", locale_translations_path(locale), class: "text-indigo-600 hover:text-indigo-900" %>
     <% end %>
   </td>
 </tr>
diff --git a/...ranslation_keys/_translation_key.html.erb → ...es/translations/_translation_key.html.erb b/...ranslation_keys/_translation_key.html.erb → ...es/translations/_translation_key.html.erb
diff --git a/...a/locales/translation_keys/index.html.erb → ...setta/locales/translations/index.html.erb b/...a/locales/translation_keys/index.html.erb → ...setta/locales/translations/index.html.erb
@@ -35,7 +35,7 @@
       </thead>
 
       <tbody class="divide-y divide-gray-200 bg-white">
-        <%= render collection: @translation_keys, partial: "rosetta/locales/translation_keys/translation_key" %>
+        <%= render collection: @translation_keys, partial: "rosetta/locales/translations/translation_key" %>
       </tbody>
     </table>
   </div>

diff --git a/app/views/rosetta/translations/edit.html.erb b/app/views/rosetta/translations/edit.html.erb
@@ -14,7 +14,7 @@
     </div>
 
     <div class="absolute inset-x-0 bottom-0 flex justify-end gap-2 py-2 pl-3 pr-2">
-      <%= link_to "Discard", locale_translation_keys_path(@locale), class: "btn btn-secondary" %>
+      <%= link_to "Discard", locale_translations_path(@locale), class: "btn btn-secondary" %>
       <%= f.submit "Save", class: "inline btn btn-primary" %>
     </div>
   <% end %>

diff --git a/config/routes.rb b/config/routes.rb
@@ -3,7 +3,7 @@
 
   resources :locales do
     scope module: :locales do
-      resources :translation_keys, only: :index
+      resources :translations, only: :index
     end
   end
 

diff --git a/...cales/translation_keys_controller_test.rb → ...a/locales/translations_controller_test.rb b/...cales/translation_keys_controller_test.rb → ...a/locales/translations_controller_test.rb
@@ -1,13 +1,13 @@
 require "test_helper"
 
 module Rosetta
-  class Locales::TranslationKeysControllerTest < ActionDispatch::IntegrationTest
+  class Locales::TranslationsControllerTest < ActionDispatch::IntegrationTest
     include Engine.routes.url_helpers
 
     test "index" do
       locale = Locale.create(code: "fr", name: "French")
       key = TranslationKey.create(value: "hello")
-      get locale_translation_keys_path(locale)
+      get locale_translations_path(locale)
       assert_response :success
       assert_includes response.body, "hello"
     end

diff --git a/test/controllers/rosetta/translations_controller_test.rb b/test/controllers/rosetta/translations_controller_test.rb
@@ -21,7 +21,7 @@ class TranslationsControllerTest < ActionDispatch::IntegrationTest
 
       assert_equal "Bonjour", Translation.last.value
       assert_response :redirect
-      assert_redirected_to locale_translation_keys_path(@locale)
+      assert_redirected_to locale_translations_path(@locale)
     end
 
     test "update an existing translation" do
@@ -33,7 +33,7 @@ class TranslationsControllerTest < ActionDispatch::IntegrationTest
 
       assert_equal "Bonjour", Translation.last.value
       assert_response :redirect
-      assert_redirected_to locale_translation_keys_path(@locale)
+      assert_redirected_to locale_translations_path(@locale)
     end
   end
 end