Skip to content

Commit cffa5e3

Browse files
committed
refactor: update addImageButtonConfig to addImageButtonClick
1 parent 596461a commit cffa5e3

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

src/components/app/details/triggerView/workflow/Workflow.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ export class Workflow extends Component<WorkflowProps> {
124124
key={`webhook-${node.id}`}
125125
id={node.id}
126126
isTemplateView={false}
127-
addImageButtonConfig={{ show: true, onClick: this.props.handleWebhookAddImageClick }}
127+
addImageButtonClick={this.props.handleWebhookAddImageClick}
128128
/>
129129
)
130130
}

src/components/workflowEditor/nodes/WebhookNode.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,14 +47,14 @@ export const WebhookNode = ({
4747
isLastNode,
4848
isReadonlyView = false,
4949
isTemplateView,
50-
addImageButtonConfig,
50+
addImageButtonClick,
5151
}: WebhookNodeProps) => {
5252
const [isWebhookTippyOpen, setIsWebhookTippyOpen] = useState(false)
5353

5454
const selectedNodeKey = `${selectedNode?.nodeType}-${selectedNode?.id}`
5555
const currentNodeKey = `${WorkflowNodeType.WEBHOOK}-${id ?? ''}`
5656

57-
const showWebhookAddImageButton = WebhookAddImageButton && (addImageButtonConfig?.show || false)
57+
const showWebhookAddImageButton = WebhookAddImageButton && !!addImageButtonClick
5858

5959
const addNewCD = (event): void => {
6060
event.preventDefault()
@@ -78,7 +78,7 @@ export const WebhookNode = ({
7878
setIsWebhookTippyOpen((prev) => !prev)
7979
}
8080

81-
const onWebhookAddImageClick = () => addImageButtonConfig?.onClick(id)
81+
const onWebhookAddImageClick = () => addImageButtonClick(id)
8282

8383
const renderWrapWithLinkOrTippy = (children: ReactElement) =>
8484
isTemplateView ? (

src/components/workflowEditor/types.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -204,10 +204,7 @@ export interface WebhookNodeProps extends Required<Pick<AppConfigProps, 'isTempl
204204
* @default false
205205
*/
206206
isReadonlyView?: boolean
207-
addImageButtonConfig?: {
208-
show: boolean
209-
onClick?: (webhookId: number) => void
210-
}
207+
addImageButtonClick?: (webhookId: number) => void
211208
}
212209

213210
export interface WebhookTippyType {

0 commit comments

Comments
 (0)