Merge branch 'main' into update-alert

KyberNetwork · Jul 19, 2023 · b4a3c60 · b4a3c60
2 parents ede008f + 26bfd05
commit b4a3c60
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/components/SubscribeButton/NotificationPreference/index.tsx b/src/components/SubscribeButton/NotificationPreference/index.tsx
@@ -175,8 +175,6 @@ function NotificationPreference({
 
   const [selectedTopic, setSelectedTopic] = useState<number[]>([])
 
-  const notFillEmail = !inputEmail
-
   const updateTopicGroupsLocal = useCallback(
     (subIds: number[], unsubIds: number[]) => {
       const newTopicGroups = topicGroupsGlobal.map(group => {
@@ -295,7 +293,7 @@ function NotificationPreference({
   const isVerifiedEmail = userInfo?.email && inputEmail === userInfo?.email
   const needVerifyEmail = inputEmail && inputEmail !== userInfo?.email
 
-  const disableCheckbox = needVerifyEmail || notFillEmail || hasErrorInput
+  const disableCheckbox = hasErrorInput
   const isIncludePriceAlert = useCallback(() => {
     const changedData = getDiffChangeTopics(topicGroups)
     return (
@@ -315,7 +313,7 @@ function NotificationPreference({
 
   const checkProfileAndSave = () => {
     if (disableButtonSave) return
-    if (!userInfo?.email && !isIncludePriceAlert()) {
+    if (needVerifyEmail && !isIncludePriceAlert()) {
       showVerifyModal()
       return
     }