Skip to content
Snippets Groups Projects
Unverified Commit db52a130 authored by Alexander Polyankin's avatar Alexander Polyankin Committed by GitHub
Browse files

Fix removing the field type in data reference (#22367)

parent f8a285c0
Branches
Tags
No related merge requests found
......@@ -44,7 +44,11 @@ const Field = ({ field, foreignKeys, url, icon, isEditing, formField }) => (
{isEditing ? (
<Select
placeholder={t`Select a field type`}
value={formField.semantic_type.value || field.semantic_type}
value={
formField.semantic_type.value !== ""
? formField.semantic_type.value
: field.semantic_type
}
onChange={({ target: { value } }) =>
formField.semantic_type.onChange(value)
}
......
import { popover, restore } from "__support__/e2e/cypress";
describe("issue 5276", () => {
beforeEach(() => {
restore();
cy.signInAsAdmin();
cy.intercept("PUT", "/api/field/*").as("updateField");
});
it("should allow removing the field type (metabase#5276)", () => {
cy.visit("/reference/databases");
cy.findByText("Sample Database").click();
cy.findByText("Tables in Sample Database").click();
cy.findByText("Products").click();
cy.findByText("Fields in this table").click();
cy.findByText("Edit").click();
cy.findByText("Score").click();
popover().within(() => cy.findByText("No field type").click());
cy.button("Save").click();
cy.wait("@updateField");
cy.findByText("Score").should("not.exist");
});
});
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment