propagate locale between group form tabs

This is a UX improvement. However, it also fixes a bug where some users were getting an error message on these pages for the locale param being an empty string (`?locale=`). Strangely, this only appeared to be affecting users using Windows, but that may not be causation.
notch8 · Jun 28, 2024 · 6bb1f07 · 6bb1f07
1 parent 3f8a5e5
commit 6bb1f07
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/app/presenters/hyku/admin/group/navigation_presenter.rb b/app/presenters/hyku/admin/group/navigation_presenter.rb
@@ -27,7 +27,7 @@ def edit_tab
               name: I18n.t('hyku.admin.groups.nav.attributes'),
               controller: 'admin/groups',
               action: 'edit',
-              path: Rails.application.routes.url_helpers.edit_admin_group_path(group_id),
+              path: Rails.application.routes.url_helpers.edit_admin_group_path(group_id, locale: I18n.locale),
               context: params
             )
           end
@@ -37,7 +37,7 @@ def members_tab
               name: I18n.t('hyku.admin.groups.nav.members'),
               controller: 'admin/group_users',
               action: 'index',
-              path: Rails.application.routes.url_helpers.admin_group_users_path(group_id),
+              path: Rails.application.routes.url_helpers.admin_group_users_path(group_id, locale: I18n.locale),
               context: params
             )
           end
@@ -47,7 +47,7 @@ def roles_tab
               name: I18n.t('hyku.admin.groups.nav.roles'),
               controller: 'admin/group_roles',
               action: 'index',
-              path: Rails.application.routes.url_helpers.admin_group_roles_path(group_id),
+              path: Rails.application.routes.url_helpers.admin_group_roles_path(group_id, locale: I18n.locale),
               context: params
             )
           end
@@ -57,7 +57,7 @@ def remove_tab
               name: I18n.t('hyku.admin.groups.nav.delete'),
               controller: 'admin/groups',
               action: 'remove',
-              path: Rails.application.routes.url_helpers.remove_admin_group_path(group_id),
+              path: Rails.application.routes.url_helpers.remove_admin_group_path(group_id, locale: I18n.locale),
               context: params
             )
           end