bringing in the metrics display toggle from the release branch

frontend/projects/upgrade/src/app/app.module.ts

@@ -33,6 +33,8 @@ export const getEnvironmentConfig = (http: HttpClient, env: Environment) => {
         env.withinSubjectExperimentSupportToggle =
           config.withinSubjectExperimentSupportToggle ?? env.withinSubjectExperimentSupportToggle ?? false;
         env.errorLogsToggle = config.errorLogsToggle ?? env.errorLogsToggle ?? false;
+        env.metricAnalyticsExperimentDisplayToggle =
+          config.metricAnalyticsExperimentDisplayToggle ?? env.metricAnalyticsExperimentDisplayToggle ?? false;
       })
       .catch((error) => {
         console.log({ error });

frontend/projects/upgrade/src/app/core/analysis/analysis.service.spec.ts

@@ -8,6 +8,7 @@ import {
   actionUpsertMetrics,
 } from './store/analysis.actions';
 import { UpsertMetrics } from './store/analysis.models';
+import { Environment } from '../../../environments/environment-types';
 
 const mockStateStore$ = new BehaviorSubject({});
 (mockStateStore$ as any).dispatch = jest.fn();
@@ -22,10 +23,12 @@ jest.mock('./store/analysis.selectors', () => ({
 
 describe('AnalysisService', () => {
   const mockStore: any = mockStateStore$;
+  let mockEnvironment: Environment = { metricAnalyticsExperimentDisplayToggle: true } as Environment;
   let service: AnalysisService;
 
   beforeEach(() => {
-    service = new AnalysisService(mockStore);
+    service = new AnalysisService(mockStore, mockEnvironment);
+    jest.resetAllMocks();
   });
 
   describe('#queryResultById$', () => {
@@ -78,12 +81,34 @@ describe('AnalysisService', () => {
   });
 
   describe('#executeQuery', () => {
-    it('should dispatch executeQuery with the supplied string input array', () => {
+    let originalEnvironment;
+
+    beforeEach(() => {
+      // Save the original environment to restore it after tests
+      originalEnvironment = { ...mockEnvironment };
+    });
+
+    afterEach(() => {
+      // Restore the original environment after each test
+      mockEnvironment = { ...originalEnvironment };
+    });
+
+    it('should dispatch executeQuery with the supplied string input array when metricAnalyticsExperimentDisplayToggle is true', () => {
+      mockEnvironment = { metricAnalyticsExperimentDisplayToggle: true } as Environment;
       const mockQueryIds = ['test', 'test2'];
 
       service.executeQuery(mockQueryIds);
 
-      expect(mockStore.dispatch).toHaveBeenLastCalledWith(actionExecuteQuery({ queryIds: mockQueryIds }));
+      expect(mockStore.dispatch).toHaveBeenCalledWith(actionExecuteQuery({ queryIds: mockQueryIds }));
+    });
+
+    it('should not dispatch executeQuery and log a warning when metricAnalyticsExperimentDisplayToggle is false', () => {
+      mockEnvironment = { metricAnalyticsExperimentDisplayToggle: false } as Environment;
+      const mockQueryIds = ['test3', 'test4'];
+
+      service.executeQuery(mockQueryIds);
+
+      expect(mockStore.dispatch).not.toHaveBeenCalledWith(mockQueryIds);
     });
   });
 

frontend/projects/upgrade/src/app/core/analysis/analysis.service.ts

@@ -1,4 +1,4 @@
-import { Injectable } from '@angular/core';
+import { Inject, Injectable } from '@angular/core';
 import { AppState } from '../core.module';
 import { Store, select } from '@ngrx/store';
 import {
@@ -11,10 +11,11 @@ import {
 import * as AnalysisActions from './store/analysis.actions';
 import { UpsertMetrics } from './store/analysis.models';
 import { selectExperimentQueries } from '../experiments/store/experiments.selectors';
+import { ENV, Environment } from '../../../environments/environment-types';
 
 @Injectable()
 export class AnalysisService {
-  constructor(private store$: Store<AppState>) {}
+  constructor(private store$: Store<AppState>, @Inject(ENV) private environment: Environment) {}
 
   isMetricsLoading$ = this.store$.pipe(select(selectIsMetricsLoading));
   isQueryExecuting$ = this.store$.pipe(select(selectIsQueryExecuting));
@@ -35,7 +36,14 @@ export class AnalysisService {
   }
 
   executeQuery(queryIds: string[]) {
-    this.store$.dispatch(AnalysisActions.actionExecuteQuery({ queryIds }));
+    if (this.environment.metricAnalyticsExperimentDisplayToggle) {
+      this.store$.dispatch(AnalysisActions.actionExecuteQuery({ queryIds }));
+    } else {
+      console.warn(
+        'executeQuery is currently disabled via metricAnalyticsExperimentDisplayToggle:',
+        this.environment.metricAnalyticsExperimentDisplayToggle
+      );
+    }
   }
 
   experimentQueryResult$(experimentId: string) {

frontend/projects/upgrade/src/app/features/dashboard/home/pages/view-experiment/view-experiment.component.html

@@ -952,7 +952,18 @@
           <ng-container *ngIf="experiment.queries.length">
             <div class="box-shadow"></div>
             <!-- Experiment queries results -->
-            <home-experiment-query-result [experiment]="experiment"></home-experiment-query-result>
+            <ng-container *ngIf="showMetricAnalysisDisplay; else noMetricsAnalysisView">
+              <home-experiment-query-result [experiment]="experiment"></home-experiment-query-result>
+            </ng-container>
+            <ng-template #noMetricsAnalysisView>
+              <div class="disabled-analysis-view-messsage-container">
+                <mat-icon class="warning-icon" color="warn">warning</mat-icon>
+                <p class="message">
+                  {{ 'home.view-experiment.metrics-analysis-unavailable.text' | translate }}
+                </p>
+              </div>
+              <div class="box-shadow"></div>
+            </ng-template>
           </ng-container>
         </div>
       </mat-tab>

frontend/projects/upgrade/src/app/features/dashboard/home/pages/view-experiment/view-experiment.component.scss

@@ -389,6 +389,17 @@ $font-size-small: 15px;
     margin-bottom: 24px;
     padding: 10px;
   }
+
+  .disabled-analysis-view-messsage-container {
+    padding-top: 50px;
+    padding-bottom: 20px;
+    display: flex;
+    justify-content: center;
+
+    .mat-icon {
+      margin-right: 20px;
+    }
+  }
 }
 
 ::ng-deep .owl-dt-calendar-table .owl-dt-calendar-cell-selected {

frontend/projects/upgrade/src/app/features/dashboard/home/pages/view-experiment/view-experiment.component.ts

@@ -1,4 +1,4 @@
-import { Component, OnInit, OnDestroy, ViewEncapsulation } from '@angular/core';
+import { Component, OnInit, OnDestroy, ViewEncapsulation, Inject } from '@angular/core';
 import { ExperimentService } from '../../../../../core/experiments/experiments.service';
 import { MatDialog } from '@angular/material/dialog';
 import { ExperimentStatusComponent } from '../../components/modal/experiment-status/experiment-status.component';
@@ -32,6 +32,7 @@ import {
   FactorialConditionTableDataFromConditionPayload,
   SimpleExperimentPayloadTableRowData,
 } from '../../../../../core/experiment-design-stepper/store/experiment-design-stepper.model';
+import { ENV, Environment } from '../../../../../../environments/environment-types';
 // Used in view-experiment component only
 enum DialogType {
   CHANGE_STATUS = 'Change status',
@@ -101,7 +102,8 @@ export class ViewExperimentComponent implements OnInit, OnDestroy {
     private dialog: MatDialog,
     private authService: AuthService,
     private router: Router,
-    private _Activatedroute: ActivatedRoute
+    private _Activatedroute: ActivatedRoute,
+    @Inject(ENV) private environment: Environment
   ) {}
 
   get DialogType() {
@@ -124,6 +126,10 @@ export class ViewExperimentComponent implements OnInit, OnDestroy {
     return this.experiment.state === EXPERIMENT_STATE.CANCELLED;
   }
 
+  get showMetricAnalysisDisplay() {
+    return this.environment.metricAnalyticsExperimentDisplayToggle;
+  }
+
   ngOnInit() {
     this.isLoadingExperimentDetailStats$ = this.experimentService.isLoadingExperimentDetailStats$;
     this.isPollingExperimentDetailStats$ = this.experimentService.isPollingExperimentDetailStats$;

frontend/projects/upgrade/src/assets/i18n/en.json

@@ -247,6 +247,7 @@
   "home.view-experiment.experiment-payloads.column-label.target": "TARGET",
   "home.view-experiment.experiment-payloads.column-label.condition": "CONDITION",
   "home.view-experiment.experiment-payloads.column-label.payload": "PAYLOAD",
+  "home.view-experiment.metrics-analysis-unavailable.text": "Experiment metrics analysis has been temporarily disabled in this view. This information is still available when exporting the experiment data.",
   "home.view-experiment.graph-type.text": "Type",
   "home.view-experiment.graph-conditions.text": "Conditions",
   "home.view-experiment.graph-decision-point.text": "Sites",

frontend/projects/upgrade/src/environments/environment-types.ts

@@ -59,8 +59,9 @@ export interface Environment {
   pollingLimit: number;
   api: APIEndpoints;
   featureFlagNavToggle: boolean;
-  withinSubjectExperimentSupportToggle: boolean;
   errorLogsToggle: boolean;
+  withinSubjectExperimentSupportToggle: boolean;
+  metricAnalyticsExperimentDisplayToggle: boolean;
 }
 
 export interface RuntimeEnvironmentConfig {
@@ -71,4 +72,5 @@ export interface RuntimeEnvironmentConfig {
   featureFlagNavToggle?: boolean;
   withinSubjectExperimentSupportToggle?: boolean;
   errorLogsToggle?: boolean;
+  metricAnalyticsExperimentDisplayToggle?: boolean;
 }

frontend/projects/upgrade/src/environments/environment.bsnl.ts

@@ -13,6 +13,7 @@ export const environment = {
   featureFlagNavToggle: false,
   withinSubjectExperimentSupportToggle: false,
   errorLogsToggle: false,
+  metricAnalyticsExperimentDisplayToggle: true,
   api: {
     getAllExperiments: '/experiments/paginated',
     createNewExperiments: '/experiments',

frontend/projects/upgrade/src/environments/environment.demo.prod.ts

@@ -13,6 +13,7 @@ export const environment = {
   featureFlagNavToggle: false,
   withinSubjectExperimentSupportToggle: false,
   errorLogsToggle: false,
+  metricAnalyticsExperimentDisplayToggle: false,
   api: {
     getAllExperiments: '/experiments/paginated',
     createNewExperiments: '/experiments',

frontend/projects/upgrade/src/environments/environment.prod.ts

@@ -13,6 +13,7 @@ export const environment = {
   featureFlagNavToggle: false,
   withinSubjectExperimentSupportToggle: false,
   errorLogsToggle: false,
+  metricAnalyticsExperimentDisplayToggle: false,
   api: {
     getAllExperiments: '/experiments/paginated',
     createNewExperiments: '/experiments',

frontend/projects/upgrade/src/environments/environment.staging.ts

@@ -13,6 +13,7 @@ export const environment = {
   featureFlagNavToggle: false,
   withinSubjectExperimentSupportToggle: false,
   errorLogsToggle: false,
+  metricAnalyticsExperimentDisplayToggle: false,
   api: {
     getAllExperiments: '/experiments/paginated',
     createNewExperiments: '/experiments',

frontend/projects/upgrade/src/environments/environment.ts

@@ -18,6 +18,7 @@ export const environment = {
   featureFlagNavToggle: true,
   withinSubjectExperimentSupportToggle: false,
   errorLogsToggle: false,
+  metricAnalyticsExperimentDisplayToggle: false,
   api: {
     getAllExperiments: '/experiments/paginated',
     createNewExperiments: '/experiments',