Skip to content

Commit

Permalink
Dashboards: Remove default empty string from variable create view (gr…
Browse files Browse the repository at this point in the history
…afana#98922)

* default empty string

* optimize
  • Loading branch information
yincongcyincong authored Feb 3, 2025
1 parent e015152 commit 6496705
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion e2e/dashboards-suite/new-query-variable.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ describe('Variables - Query - Add variable', () => {
cy.get('input[type="checkbox"]').should('not.be.checked');
});

e2e.pages.Dashboard.Settings.Variables.Edit.General.previewOfValuesOption().should('not.have.text');
e2e.pages.Dashboard.Settings.Variables.Edit.General.previewOfValuesOption().should('not.exist');
e2e.pages.Dashboard.Settings.Variables.Edit.General.selectionOptionsCustomAllInput().should('not.exist');
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ export function VariableEditorForm({

{EditorToRender && <EditorToRender variable={variable} onRunQuery={onRunQuery} />}

{isHasVariableOptions && <VariableValuesPreview options={variable.getOptionsForSelect()} />}
{isHasVariableOptions && <VariableValuesPreview options={variable.getOptionsForSelect(false)} />}

<div className={styles.buttonContainer}>
<Stack gap={2}>
Expand Down

0 comments on commit 6496705

Please sign in to comment.