From b4f6f9f008a1aafe5de1412bc0c3df744e1ff9a5 Mon Sep 17 00:00:00 2001
From: Lorenzo Blasa <lblasa@meta.com>
Date: Fri, 10 Mar 2023 06:18:21 -0800
Subject: [PATCH] TimelineDataDescription optional onClick

Summary: Make this property optional as is not always required to implement it, if not needed.

Reviewed By: mweststrate

Differential Revision: D43948425

fbshipit-source-id: 43ed761c70c4779134f277a4cda53c1e3d326cb8
---
 .../src/ui/data-inspector/TimelineDataDescription.tsx         | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/desktop/flipper-plugin/src/ui/data-inspector/TimelineDataDescription.tsx b/desktop/flipper-plugin/src/ui/data-inspector/TimelineDataDescription.tsx
index c3bc1c3f8..24fb34fb1 100644
--- a/desktop/flipper-plugin/src/ui/data-inspector/TimelineDataDescription.tsx
+++ b/desktop/flipper-plugin/src/ui/data-inspector/TimelineDataDescription.tsx
@@ -30,7 +30,7 @@ type Timeline = {
 type Props = {
   canSetCurrent?: boolean;
   timeline: Timeline;
-  onClick: (selected: string) => void;
+  onClick?: (selected: string) => void;
 };
 
 type State = {
@@ -59,7 +59,7 @@ export class TimelineDataDescription extends Component<Props, State> {
         {this.props.canSetCurrent && (
           <div>
             <Button
-              onClick={() => this.props.onClick(this.state.selected)}
+              onClick={() => this.props.onClick?.(this.state.selected)}
               disabled={this.state.selected === this.props.timeline.current}>
               Set as current
             </Button>