chore(settings): remove defaultCategory from settings.js (#7550)

It seems this setting is no longer used by any code
spinnaker · Oct 23, 2019 · 2d78431 · 2d78431
1 parent a71f76d
commit 2d78431
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/app/scripts/modules/core/src/config/settings.ts b/app/scripts/modules/core/src/config/settings.ts
@@ -94,7 +94,6 @@ export interface ISpinnakerSettings {
   };
   checkForUpdates: boolean;
   debugEnabled: boolean;
-  defaultCategory: string;
   defaultInstancePort: number;
   defaultProviders: string[];
   defaultTimeZone: string; // see http://momentjs.com/timezone/docs/#/data-utilities/

diff --git a/halconfig/settings.js b/halconfig/settings.js
@@ -136,7 +136,6 @@ window.spinnakerSettings = {
     gistId: changelogGistId,
   },
   checkForUpdates: false,
-  defaultCategory: 'serverGroup',
   defaultInstancePort: 80,
   defaultProviders: [
     'appengine',

diff --git a/settings.js b/settings.js
@@ -1,5 +1,7 @@
 'use strict';
 
+// Use environment variables when developing locqlly via 'yarn start', i.e.:
+// API_HOST=https://gate.spinnaker.mycompany.com yarn start
 var apiHost = process.env.API_HOST || 'http://localhost:8084';
 var artifactsEnabled = process.env.ARTIFACTS_ENABLED === 'true';
 var artifactsRewriteEnabled = process.env.ARTIFACTS_REWRITE_ENABLED === 'true';
@@ -50,7 +52,6 @@ window.spinnakerSettings = {
   },
   checkForUpdates: true,
   debugEnabled: debugEnabled,
-  defaultCategory: 'serverGroup',
   defaultInstancePort: 80,
   defaultProviders: ['appengine', 'aws', 'azure', 'cloudfoundry', 'dcos', 'ecs', 'gce', 'kubernetes', 'oracle'],
   defaultTimeZone: process.env.TIMEZONE || 'America/Los_Angeles', // see http://momentjs.com/timezone/docs/#/data-utilities/