Skip to content

Commit

Permalink
Removing monitoring
Browse files Browse the repository at this point in the history
Logging had already been removed from the navigation menu but the translation was still present. I removed the translation for completeness.

I left the routes and pages just in case we need to allow the customer to visit those pages even if it's unlikely. We will eventually fully remove ember so I don't think there's a reason to tidy up too much in this scenario.

rancher/dashboard#10617
  • Loading branch information
codyrancher committed Apr 22, 2024
1 parent a9ca59c commit d81fd05
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
19 changes: 0 additions & 19 deletions app/instance-initializers/nav.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,14 +106,6 @@ const rootNav = [
resource: ['catalog', 'project-catalog'],
resourceScope: 'global',
},
{
id: 'tools-monitoring',
localizedLabel: 'nav.tools.monitoring',
route: 'authenticated.project.monitoring.project-setting',
resourceScope: 'global',
resource: [],
ctx: [getProjectId],
},
{
id: 'tools-pipeline',
localizedLabel: 'nav.tools.pipeline',
Expand Down Expand Up @@ -233,17 +225,6 @@ const rootNav = [
resource: [],
ctx: [getClusterId],
},
{
id: 'cluster-tools-monitoring',
localizedLabel: 'nav.tools.monitoring',
route: 'authenticated.cluster.monitoring.cluster-setting',
resourceScope: 'global',
resource: [],
ctx: [getClusterId],
condition() {
return get(this, 'cluster.isActive')
},
},
{ divider: true },
{
id: 'cluster-tools-backup',
Expand Down
2 changes: 0 additions & 2 deletions translations/en-us.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10461,8 +10461,6 @@ nav:
alerts: Alerts
backups: Snapshots
catalogs: Catalogs
logging: Logging
monitoring: Monitoring
notifiers: Notifiers
pipeline: Pipeline
tab: Tools
Expand Down

0 comments on commit d81fd05

Please sign in to comment.