Connections: Rename "Your data source" to "Data source" page (#67662)

rename "Your data source" to "Data source" page

This page was meant to be named "Data source" in the previous PR [0].
I just made a mistake by naming it wrongly.

[0]: https://github.com/grafana/grafana/pull/66813/files
This commit is contained in:
mikkancso 2023-05-03 09:39:13 +02:00 committed by GitHub
parent 7ed93cea54
commit 76284ed3a6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
9 changed files with 32 additions and 32 deletions

View File

@ -126,9 +126,9 @@ func (hs *HTTPServer) registerRoutes() {
r.Get("/plugins/:id/edit", middleware.CanAdminPlugins(hs.Cfg), hs.Index) // deprecated r.Get("/plugins/:id/edit", middleware.CanAdminPlugins(hs.Cfg), hs.Index) // deprecated
r.Get("/plugins/:id/page/:page", middleware.CanAdminPlugins(hs.Cfg), hs.Index) r.Get("/plugins/:id/page/:page", middleware.CanAdminPlugins(hs.Cfg), hs.Index)
r.Get("/connections/your-datasources", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index) r.Get("/connections/datasources", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index)
r.Get("/connections/your-datasources/new", authorize(reqOrgAdmin, datasources.NewPageAccess), hs.Index) r.Get("/connections/datasources/new", authorize(reqOrgAdmin, datasources.NewPageAccess), hs.Index)
r.Get("/connections/your-datasources/edit/*", authorize(reqOrgAdmin, datasources.EditPageAccess), hs.Index) r.Get("/connections/datasources/edit/*", authorize(reqOrgAdmin, datasources.EditPageAccess), hs.Index)
r.Get("/connections", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index) r.Get("/connections", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index)
r.Get("/connections/add-new-connection", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index) r.Get("/connections/add-new-connection", authorize(reqOrgAdmin, datasources.ConfigurationPageAccess), hs.Index)
r.Get("/connections/datasources/:id", middleware.CanAdminPlugins(hs.Cfg), hs.Index) r.Get("/connections/datasources/:id", middleware.CanAdminPlugins(hs.Cfg), hs.Index)

View File

@ -490,12 +490,12 @@ func (s *ServiceImpl) buildDataConnectionsNavLink(c *contextmodel.ReqContext) *n
Children: []*navtree.NavLink{}, Children: []*navtree.NavLink{},
}) })
// Your data sources // Data sources
children = append(children, &navtree.NavLink{ children = append(children, &navtree.NavLink{
Id: "connections-your-datasources", Id: "connections-datasources",
Text: "Your data sources", Text: "Data sources",
SubTitle: "View and manage your connected data source connections", SubTitle: "View and manage your connected data source connections",
Url: baseUrl + "/your-datasources", Url: baseUrl + "/datasources",
Children: []*navtree.NavLink{}, Children: []*navtree.NavLink{},
}) })
} }

View File

@ -55,7 +55,7 @@ describe('Connections', () => {
expect(await screen.findByText('Add new connection')).toBeVisible(); expect(await screen.findByText('Add new connection')).toBeVisible();
expect(await screen.findByText('Browse and create new connections')).toBeVisible(); expect(await screen.findByText('Browse and create new connections')).toBeVisible();
// Should not render the "Your datasources" page // Should not render the "datasources" page
expect(screen.queryByText('Manage your existing datasource connections')).not.toBeInTheDocument(); expect(screen.queryByText('Manage your existing datasource connections')).not.toBeInTheDocument();
}); });

View File

@ -235,15 +235,15 @@ export const navIndex: NavIndex = {
url: '/connections/add-new-connection', url: '/connections/add-new-connection',
}, },
{ {
id: 'connections-your-datasources', id: 'connections-datasources',
text: 'Your data sources', text: 'Data sources',
subTitle: 'Manage your existing datasource connections', subTitle: 'Manage your existing datasource connections',
url: '/connections/your-datasources', url: '/connections/datasources',
}, },
{ {
id: 'standalone-plugin-page-/connections/your-infrastructure', id: 'standalone-plugin-page-/connections/infrastructure',
text: 'Your infrastructure', text: 'Infrastructure',
url: '/connections/your-infrastructure', url: '/connections/infrastructure',
pluginId: 'grafana-easystart-app', pluginId: 'grafana-easystart-app',
}, },
], ],
@ -255,16 +255,16 @@ export const navIndex: NavIndex = {
sortWeight: -2000, sortWeight: -2000,
}, },
}, },
'connections-your-datasources': { 'connections-datasources': {
id: 'connections-your-datasources', id: 'connections-datasources',
text: 'Your data sources', text: 'Data sources',
subTitle: 'Manage your existing datasource connections', subTitle: 'Manage your existing datasource connections',
url: '/connections/your-datasources', url: '/connections/datasources',
}, },
'standalone-plugin-page-/connections/your-infrastructure': { 'standalone-plugin-page-/connections/infrastructure': {
id: 'standalone-plugin-page-/connections/your-infrastructure', id: 'standalone-plugin-page-/connections/infrastructure',
text: 'Your infrastructure', text: 'Infrastructure',
url: '/connections/your-infrastructure', url: '/connections/infrastructure',
pluginId: 'grafana-easystart-app', pluginId: 'grafana-easystart-app',
}, },
'connections-add-new-connection': { 'connections-add-new-connection': {

View File

@ -4,11 +4,11 @@ export const ROUTE_BASE_ID = 'connections';
export const ROUTES = { export const ROUTES = {
Base: `/${ROUTE_BASE_ID}`, Base: `/${ROUTE_BASE_ID}`,
// Your Datasources // Data sources
DataSources: `/${ROUTE_BASE_ID}/your-datasources`, DataSources: `/${ROUTE_BASE_ID}/datasources`,
DataSourcesNew: `/${ROUTE_BASE_ID}/your-datasources/new`, DataSourcesNew: `/${ROUTE_BASE_ID}/datasources/new`,
DataSourcesEdit: `/${ROUTE_BASE_ID}/your-datasources/edit/:uid`, DataSourcesEdit: `/${ROUTE_BASE_ID}/datasources/edit/:uid`,
DataSourcesDashboards: `/${ROUTE_BASE_ID}/your-datasources/edit/:uid/dashboards`, DataSourcesDashboards: `/${ROUTE_BASE_ID}/datasources/edit/:uid/dashboards`,
// Add new connection // Add new connection
AddNewConnection: `/${ROUTE_BASE_ID}/add-new-connection`, AddNewConnection: `/${ROUTE_BASE_ID}/add-new-connection`,

View File

@ -23,12 +23,12 @@ export function useDataSourceSettingsNav(pageId?: string) {
active: true, active: true,
children: (nav.main.children || []).map((navModelItem) => ({ children: (nav.main.children || []).map((navModelItem) => ({
...navModelItem, ...navModelItem,
url: navModelItem.url?.replace('datasources/edit/', '/connections/your-datasources/edit/'), url: navModelItem.url?.replace('datasources/edit/', '/connections/datasources/edit/'),
})), })),
}; };
return { return {
navId: 'connections-your-datasources', navId: 'connections-datasources',
pageNav, pageNav,
}; };
} }

View File

@ -11,7 +11,7 @@ export function DataSourcesListPage() {
const actions = dataSourcesCount > 0 ? <DataSourceAddButton /> : undefined; const actions = dataSourcesCount > 0 ? <DataSourceAddButton /> : undefined;
return ( return (
<Page navId={'connections-your-datasources'} actions={actions}> <Page navId={'connections-datasources'} actions={actions}>
<Page.Contents> <Page.Contents>
<DataSourcesList /> <DataSourcesList />
</Page.Contents> </Page.Contents>

View File

@ -6,7 +6,7 @@ import { NewDataSource } from 'app/features/datasources/components/NewDataSource
export function NewDataSourcePage() { export function NewDataSourcePage() {
return ( return (
<Page <Page
navId={'connections-your-datasources'} navId={'connections-datasources'}
pageNav={{ text: 'Add data source', subTitle: 'Choose a data source type', active: true }} pageNav={{ text: 'Add data source', subTitle: 'Choose a data source type', active: true }}
> >
<Page.Contents> <Page.Contents>

View File

@ -91,7 +91,7 @@ export function NoAccessModal({ item, isOpen, onDismiss }: NoAccessModalProps) {
<div> <div>
<p> <p>
Editors cannot add new connections. You may check to see if it is already configured in{' '} Editors cannot add new connections. You may check to see if it is already configured in{' '}
<a href="/connections/your-datasources">Your data sources</a>. <a href="/connections/datasources">Data sources</a>.
</p> </p>
<p>To add a new connection, contact your Grafana admin.</p> <p>To add a new connection, contact your Grafana admin.</p>
</div> </div>