From 714a68bfdf4024854a457c4d53af37c974d9fdc6 Mon Sep 17 00:00:00 2001
From: Kyle Johnson <kyle@solidstategroup.com>
Date: Wed, 16 Oct 2024 11:28:06 +0100
Subject: [PATCH] fix: Combine segment override and value change requests
 (#4734)

---
 .../components/modals/AssociatedSegmentOverrides.js | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/frontend/web/components/modals/AssociatedSegmentOverrides.js b/frontend/web/components/modals/AssociatedSegmentOverrides.js
index 390f94d0ba45..06f48deb5c47 100644
--- a/frontend/web/components/modals/AssociatedSegmentOverrides.js
+++ b/frontend/web/components/modals/AssociatedSegmentOverrides.js
@@ -177,11 +177,7 @@ class TheComponent extends Component {
             }
             renderRow={(v) => (
               <div key={v.feature.id} className='list-item-override p-3 mb-4'>
-                <div
-                  onClick={() => {
-                    // window.open(`${document.location.origin}/project/${this.props.projectId}/environment/${v.env.api_key}/features?feature=${v.feature.id}&tab=1`)
-                  }}
-                >
+                <div>
                   <WrappedSegmentOverrides
                     onSave={this.fetch}
                     projectFlag={v.feature}
@@ -331,9 +327,10 @@ export default class SegmentOverridesInner extends Component {
               )
             this.setState({ isSaving: true })
           })
-          const segmentOverride =
-            segmentOverrides && segmentOverrides.filter((v) => v.segment === id)
-          if (!segmentOverrides) return null
+          const segmentOverride = segmentOverrides?.filter?.(
+            (v) => v.segment === id,
+          )
+          if (!segmentOverride?.length) return null
           return (
             <div>
               {originalSegmentOverrides.length > 1 && (