[Vis Builder] Update vislib params and misc fixes (#2610)

* Updates bar, line, area visbuilder config Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com> * Misc fixes from functional testing Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com> * Adds Changelog entry Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com> Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com> (cherry picked from commit 33aed0a)
opensearch-project · Oct 20, 2022 · 62a1515 · 62a1515
1 parent e9e7e97
commit 62a1515
Show file tree

Hide file tree

Showing 8 changed files with 75 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -28,6 +28,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 * [Vis Builder] Fixes auto bounds for timeseries bar chart visualization ([2401](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2401))
 * [Vis Builder] Fixes visualization shift when editing agg ([2401](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2401))
 * [Vis Builder] Renames "Histogram" to "Bar" in vis type picker ([2401](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2401))
+* [Vis Builder] Update vislib params and misc fixes ([2610](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2610))
 * [MD] Add data source param to low-level search call in Discover ([#2431](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2431))
 * [Multi DataSource] Skip data source view in index pattern step when pick default ([#2574](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2574))
 

diff --git a/src/plugins/visualize/public/application/components/visualize_listing.tsx b/src/plugins/visualize/public/application/components/visualize_listing.tsx
@@ -123,7 +123,9 @@ export const VisualizeListing = () => {
         .then(({ total, hits }: { total: number; hits: object[] }) => ({
           total,
           hits: hits.filter(
-            (result: any) => isLabsEnabled || result.type?.stage !== 'experimental'
+            (result: any) =>
+              isLabsEnabled ||
+              (result.type?.stage !== 'experimental' && result.stage !== 'experimental')
           ),
         }));
     },

diff --git a/src/plugins/wizard/public/embeddable/disabled_visualization.tsx b/src/plugins/wizard/public/embeddable/disabled_visualization.tsx
@@ -11,7 +11,7 @@ import './disabled_visualization.scss';
 
 export function DisabledVisualization({ title }: { title: string }) {
   return (
-    <div className="vbDisabledVisualization">
+    <div className="vbDisabledVisualization" data-test-subj="disabledVisBuilderVis">
       <EuiIcon type="beaker" size="xl" />
       <div>
         <FormattedMessage

diff --git a/src/plugins/wizard/public/embeddable/wizard_embeddable.tsx b/src/plugins/wizard/public/embeddable/wizard_embeddable.tsx
@@ -29,6 +29,7 @@ import {
 } from '../../../data/public';
 import { validateSchemaState } from '../application/utils/validate_schema_state';
 import { getExpressionLoader, getTypeService } from '../plugin_services';
+import { PersistedState } from '../../../visualizations/public';
 
 // Apparently this needs to match the saved object type for the clone and replace panel actions to work
 export const WIZARD_EMBEDDABLE = WIZARD_SAVED_OBJECT;
@@ -65,6 +66,7 @@ export class WizardEmbeddable extends Embeddable<SavedObjectEmbeddableInput, Wiz
   private node?: HTMLElement;
   private savedWizard?: WizardSavedObjectAttributes;
   private serializedState?: { visualization: string; style: string };
+  private uiState?: PersistedState;
 
   constructor(
     timefilter: TimefilterContract,
@@ -90,6 +92,7 @@ export class WizardEmbeddable extends Embeddable<SavedObjectEmbeddableInput, Wiz
     );
 
     this.savedWizard = savedWizard;
+    this.uiState = new PersistedState();
 
     this.autoRefreshFetchSubscription = timefilter
       .getAutoRefreshFetch$()
@@ -226,6 +229,7 @@ export class WizardEmbeddable extends Embeddable<SavedObjectEmbeddableInput, Wiz
         query: this.input.query,
         filters: this.input.filters,
       },
+      uiState: this.uiState,
     };
     if (this.abortController) {
       this.abortController.abort();

diff --git a/src/plugins/wizard/public/embeddable/wizard_embeddable_factory.tsx b/src/plugins/wizard/public/embeddable/wizard_embeddable_factory.tsx
@@ -71,11 +71,8 @@ export class WizardEmbeddableFactoryDefinition
   ): Promise<WizardEmbeddable | ErrorEmbeddable | DisabledEmbeddable> {
     try {
       const savedWizard = await getSavedWizardLoader().get(savedObjectId);
-
       const editPath = `${EDIT_PATH}/${savedObjectId}`;
-
       const editUrl = getHttp().basePath.prepend(`/app/${PLUGIN_ID}${editPath}`);
-
       const isLabsEnabled = getUISettings().get<boolean>(VISUALIZE_ENABLE_LABS_SETTING);
 
       if (!isLabsEnabled) {

diff --git a/src/plugins/wizard/public/visualizations/vislib/area/area_vis_type.ts b/src/plugins/wizard/public/visualizations/vislib/area/area_vis_type.ts
@@ -46,6 +46,28 @@ export const createAreaConfig = (): VisualizationTypeOptions<AreaOptionsDefaults
             min: 0,
             max: 1,
             aggFilter: ['!geohash_grid', '!geotile_grid', '!filter', '!filters'],
+            defaults: { aggTypes: ['date_histogram', 'terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'group',
+            title: i18n.translate('visTypeVislib.area.groupTitle', {
+              defaultMessage: 'Split series',
+            }),
+            min: 0,
+            max: 3,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
+            defaults: { aggTypes: ['terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'split',
+            title: i18n.translate('visTypeVislib.area.splitTitle', {
+              defaultMessage: 'Split chart',
+            }),
+            min: 0,
+            max: 1,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
             defaults: { aggTypes: ['terms'] },
           },
         ]),

diff --git a/src/plugins/wizard/public/visualizations/vislib/histogram/histogram_vis_type.ts b/src/plugins/wizard/public/visualizations/vislib/histogram/histogram_vis_type.ts
@@ -46,6 +46,28 @@ export const createHistogramConfig = (): VisualizationTypeOptions<HistogramOptio
             min: 0,
             max: 1,
             aggFilter: ['!geohash_grid', '!geotile_grid', '!filter', '!filters'],
+            defaults: { aggTypes: ['date_histogram', 'terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'group',
+            title: i18n.translate('visTypeVislib.histogram.groupTitle', {
+              defaultMessage: 'Split series',
+            }),
+            min: 0,
+            max: 3,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
+            defaults: { aggTypes: ['terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'split',
+            title: i18n.translate('visTypeVislib.histogram.splitTitle', {
+              defaultMessage: 'Split chart',
+            }),
+            min: 0,
+            max: 1,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
             defaults: { aggTypes: ['terms'] },
           },
         ]),

diff --git a/src/plugins/wizard/public/visualizations/vislib/line/line_vis_type.ts b/src/plugins/wizard/public/visualizations/vislib/line/line_vis_type.ts
@@ -46,6 +46,28 @@ export const createLineConfig = (): VisualizationTypeOptions<LineOptionsDefaults
             min: 0,
             max: 1,
             aggFilter: ['!geohash_grid', '!geotile_grid', '!filter', '!filters'],
+            defaults: { aggTypes: ['date_histogram', 'terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'group',
+            title: i18n.translate('visTypeVislib.line.groupTitle', {
+              defaultMessage: 'Split series',
+            }),
+            min: 0,
+            max: 3,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
+            defaults: { aggTypes: ['terms'] },
+          },
+          {
+            group: AggGroupNames.Buckets,
+            name: 'split',
+            title: i18n.translate('visTypeVislib.line.splitTitle', {
+              defaultMessage: 'Split chart',
+            }),
+            min: 0,
+            max: 1,
+            aggFilter: ['!geohash_grid', '!geotile_grid', '!filter'],
             defaults: { aggTypes: ['terms'] },
           },
           {