From 8de1047f65482ad35be9692dd3a4903e6513b559 Mon Sep 17 00:00:00 2001 From: Tim Levett Date: Tue, 1 Oct 2024 05:31:31 -0500 Subject: [PATCH] Change from Apps to "More Apps" (#93454) * Change the label used for additional app links from apps to more apps so it doesn't conflict with applications, which is application observability, not the other bucket * update to more apps * more in german is mehr * fix case, update translations correctly * revert changes to de * fix be tests --------- Co-authored-by: joshhunt Co-authored-by: Ashley Harrison --- pkg/services/navtree/navtreeimpl/applinks.go | 2 +- pkg/services/navtree/navtreeimpl/applinks_test.go | 8 ++++---- public/app/core/utils/navBarItem-translations.ts | 2 +- public/locales/en-US/grafana.json | 2 +- public/locales/pseudo-LOCALE/grafana.json | 2 +- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/pkg/services/navtree/navtreeimpl/applinks.go b/pkg/services/navtree/navtreeimpl/applinks.go index e9fb15d950d..ec7f9ee65bf 100644 --- a/pkg/services/navtree/navtreeimpl/applinks.go +++ b/pkg/services/navtree/navtreeimpl/applinks.go @@ -195,7 +195,7 @@ func (s *ServiceImpl) addPluginToSection(c *contextmodel.ReqContext, treeRoot *n switch sectionID { case navtree.NavIDApps: treeRoot.AddSection(&navtree.NavLink{ - Text: "Apps", + Text: "More apps", Icon: "layer-group", SubTitle: "App plugins that extend the Grafana experience", Id: navtree.NavIDApps, diff --git a/pkg/services/navtree/navtreeimpl/applinks_test.go b/pkg/services/navtree/navtreeimpl/applinks_test.go index fdac09b8450..40978045b44 100644 --- a/pkg/services/navtree/navtreeimpl/applinks_test.go +++ b/pkg/services/navtree/navtreeimpl/applinks_test.go @@ -121,14 +121,14 @@ func TestAddAppLinks(t *testing.T) { }, } - t.Run("Should move apps to Apps category", func(t *testing.T) { + t.Run("Should move apps to 'More apps' category", func(t *testing.T) { treeRoot := navtree.NavTreeRoot{} err := service.addAppLinks(&treeRoot, reqCtx) require.NoError(t, err) appsNode := treeRoot.FindById(navtree.NavIDApps) require.NotNil(t, appsNode) - require.Equal(t, "Apps", appsNode.Text) + require.Equal(t, "More apps", appsNode.Text) require.Len(t, appsNode.Children, 3) require.Equal(t, testApp1.Name, appsNode.Children[0].Text) }) @@ -169,7 +169,7 @@ func TestAddAppLinks(t *testing.T) { require.Len(t, treeRoot.Children, 2) require.Equal(t, "plugin-page-test-app1", treeRoot.Children[0].Id) - // Check if it is not under the "Apps" section anymore + // Check if it is not under the "More apps" section anymore appsNode := treeRoot.FindById(navtree.NavIDApps) require.NotNil(t, appsNode) require.Len(t, appsNode.Children, 2) @@ -197,7 +197,7 @@ func TestAddAppLinks(t *testing.T) { require.Len(t, adminNode.Children, 1) require.Equal(t, "plugin-page-test-app1", adminNode.Children[0].Id) - // Check if it is not under the "Apps" section anymore + // Check if it is not under the "More apps" section anymore appsNode := treeRoot.FindById(navtree.NavIDApps) require.NotNil(t, appsNode) require.Len(t, appsNode.Children, 2) diff --git a/public/app/core/utils/navBarItem-translations.ts b/public/app/core/utils/navBarItem-translations.ts index 57be9f87a5f..d5a664526af 100644 --- a/public/app/core/utils/navBarItem-translations.ts +++ b/public/app/core/utils/navBarItem-translations.ts @@ -140,7 +140,7 @@ export function getNavTitle(navId: string | undefined) { case 'frontend': return t('nav.frontend.title', 'Frontend'); case 'apps': - return t('nav.apps.title', 'Apps'); + return t('nav.apps.title', 'More apps'); case 'alerts-and-incidents': return t('nav.alerts-and-incidents.title', 'Alerts & IRM'); case 'testing-and-synthetics': diff --git a/public/locales/en-US/grafana.json b/public/locales/en-US/grafana.json index 310d59feacb..0c9b09b6853 100644 --- a/public/locales/en-US/grafana.json +++ b/public/locales/en-US/grafana.json @@ -1574,7 +1574,7 @@ }, "apps": { "subtitle": "App plugins that extend the Grafana experience", - "title": "Apps" + "title": "More apps" }, "authentication": { "title": "Authentication" diff --git a/public/locales/pseudo-LOCALE/grafana.json b/public/locales/pseudo-LOCALE/grafana.json index b8107773274..8e39227ae2f 100644 --- a/public/locales/pseudo-LOCALE/grafana.json +++ b/public/locales/pseudo-LOCALE/grafana.json @@ -1574,7 +1574,7 @@ }, "apps": { "subtitle": "Åpp pľūģįʼnş ŧĥäŧ ęχŧęʼnđ ŧĥę Ğřäƒäʼnä ęχpęřįęʼnčę", - "title": "Åppş" + "title": "Mőřę äppş" }, "authentication": { "title": "Åūŧĥęʼnŧįčäŧįőʼn"