Skip to content
Snippets Groups Projects
Unverified Commit 67921786 authored by Ryan Laurie's avatar Ryan Laurie Committed by GitHub
Browse files

Data Apps: better action-adding behavior (#25719)

* better action-adding behavior

* use viz settings lib
parent 732bf6b1
No related branches found
No related tags found
No related merge requests found
......@@ -5,6 +5,7 @@ import { connect } from "react-redux";
import Actions from "metabase/entities/actions";
import { updateButtonActionMapping } from "metabase/dashboard/actions";
import { updateSettings } from "metabase/visualizations/lib/settings";
import type {
ActionDashboardCard,
......@@ -30,6 +31,8 @@ interface ActionOptionsDispatchProps {
dashCardId: number,
settings: {
action_id?: number | null;
action?: WritebackAction;
visualization_settings?: ActionDashboardCard["visualization_settings"];
parameter_mappings?: ActionParametersMapping[] | null;
},
) => void;
......@@ -57,7 +60,11 @@ function ActionOptions({
(action: WritebackAction) => {
onUpdateButtonActionMapping(dashcard.id, {
action_id: action.id,
action,
visualization_settings: updateSettings(
{ "button.label": action.name },
dashcard.visualization_settings,
),
// Clean mappings from previous action
// as they're most likely going to be irrelevant
parameter_mappings: null,
......
import React from "react";
import { t } from "ttag";
import type { VisualizationProps } from "metabase-types/types/Visualization";
......@@ -28,7 +29,7 @@ function ActionButtonView({
isFullHeight={isFullHeight}
{...variantProps}
>
{label}
{label ?? t`Click me`}
</StyledButton>
);
}
......
......@@ -38,7 +38,6 @@ export default Object.assign(Action, {
section: t`Display`,
title: t`Label`,
widget: "input",
default: "Click Me",
},
"button.variant": {
section: t`Display`,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment