adds enable_visibility to togle package visibility settings ui

Signed-off-by: Matt Wrock <[email protected]>
habitat-sh · Jul 19, 2024 · b36c602 · b36c602
1 parent f8cbee3
commit b36c602
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 2 deletions.
diff --git a/components/builder-api-proxy/habitat/config/habitat.conf.js b/components/builder-api-proxy/habitat/config/habitat.conf.js
@@ -5,6 +5,7 @@ habitatConfig({
     demo_app_url: "{{cfg.demo_app_url}}",
     docs_url: "{{cfg.docs_url}}",
     enable_builder: {{cfg.enable_builder}},
+    enable_visibility: {{cfg.enable_visibility}},
     enable_publisher_amazon: {{cfg.enable_publisher_amazon}},
     enable_publisher_azure: {{cfg.enable_publisher_azure}},
     enable_publisher_docker: {{cfg.enable_publisher_docker}},

diff --git a/components/builder-api-proxy/habitat/default.toml b/components/builder-api-proxy/habitat/default.toml
@@ -6,6 +6,7 @@ enable_publisher_amazon   = false
 enable_publisher_azure    = false
 enable_publisher_docker   = false
 enable_builder            = false
+enable_visibility         = false
 environment               = "production"
 hosted                    = false
 source_code_url           = "https://github.com/habitat-sh/habitat"

diff --git a/components/builder-web/app/origin/origin-page/origin-page.component.html b/components/builder-web/app/origin/origin-page/origin-page.component.html
@@ -34,7 +34,7 @@ <h2>origin</h2>
       routerLinkActive
       #settings="routerLinkActive"
       [active]="settings.isActive"
-      *ngIf="memberOfOrigin">Settings</a>
+      *ngIf="visibilityEnabled && memberOfOrigin">Settings</a>
     <a mat-tab-link
       routerLink="integrations"
       routerLinkActive

diff --git a/components/builder-web/app/origin/origin-page/origin-page.component.ts b/components/builder-web/app/origin/origin-page/origin-page.component.ts
@@ -52,6 +52,10 @@ export class OriginPageComponent implements OnInit, OnDestroy {
     return this.store.getState().features.builder;
   }
 
+  get visibilityEnabled() {
+    return this.store.getState().features.visibility;
+  }
+
   get origin() {
     const current = this.store.getState().origins.current;
 

diff --git a/components/builder-web/app/package/package/package.component.html b/components/builder-web/app/package/package/package.component.html
@@ -57,7 +57,7 @@ <h2>{{ subheading }}
       Manifest
     </a>
     <a
-      *ngIf="builderEnabled && isOriginMember"
+      *ngIf="visibilityEnabled && isOriginMember"
       mat-tab-link
       [routerLink]="['./', version, release, 'settings']"
       routerLinkActive

diff --git a/components/builder-web/app/package/package/package.component.ts b/components/builder-web/app/package/package/package.component.ts
@@ -180,6 +180,10 @@ export class PackageComponent implements OnInit, OnDestroy {
     return this.store.getState().features.builder;
   }
 
+  get visibilityEnabled() {
+    return this.store.getState().features.visibility;
+  }
+
   get activePackage() {
     return this.store.getState().packages.current;
   }

diff --git a/components/builder-web/app/reducers/features.ts b/components/builder-web/app/reducers/features.ts
@@ -25,6 +25,7 @@ export default function builds(state = initialState['features'], action) {
         .setIn(['publishers', 'azure'], !!config['enable_publisher_azure'])
         .setIn(['publishers', 'docker'], !!config['enable_publisher_docker'])
         .set('builder', !!config['enable_builder'])
+        .set('visibility', !!config['enable_visibility'])
         .set('events', !!config['enable_builder_events'])
         .set('saasEvents', !!config['enable_builder_events_saas']);
     default:

diff --git a/components/builder-web/habitat.conf.sample.js b/components/builder-web/habitat.conf.sample.js
@@ -16,6 +16,9 @@ habitatConfig({
     // Enable Builder-specific features
     enable_builder: true,
 
+    // Enable ability to set project visibility
+    enable_visibility: true,
+
     // Enable supported container-registry integrations
     enable_publisher_amazon: false,
     enable_publisher_azure: false,