CodeQL fixes

Author: rajat1saxena

apps/web/app/(with-contexts)/dashboard/(sidebar)/mails/broadcast/[id]/page.tsx

@@ -72,7 +72,7 @@ export default function Page({
     const [delay, setDelay] = useState(0);
     const [showScheduleInput, setShowScheduleInput] = useState(false);
     const [emailId, setEmailId] = useState<string | undefined>();
-    const [published, setPublished] = useState(true);
+    // const [published, setPublished] = useState(true);
     const [filteredUsersCount, setFilteredUsersCount] = useState(0);
     const [confirmationDialogOpen, setConfirmationDialogOpen] = useState(false);
     const [report, setReport] = useState<SequenceReport>();
@@ -115,7 +115,7 @@ export default function Page({
             setContent(sequence.emails[0].content);
             setDelay(sequence.emails[0].delayInMillis);
             setEmailId(sequence.emails[0].emailId);
-            setPublished(sequence.emails[0].published);
+            // setPublished(sequence.emails[0].published);
             if (sequence.filter) {
                 setFilters(sequence.filter.filters);
                 setFiltersAggregator(sequence.filter.aggregator);
@@ -370,7 +370,7 @@ export default function Page({
                 setContent(sequence.emails[0].content);
                 setDelay(sequence.emails[0].delayInMillis);
                 setEmailId(sequence.emails[0].emailId);
-                setPublished(sequence.emails[0].published);
+                // setPublished(sequence.emails[0].published);
                 if (sequence.filter) {
                     setFilters(sequence.filter.filters);
                     setFiltersAggregator(sequence.filter.aggregator);
@@ -448,7 +448,7 @@ export default function Page({
                 setContent(sequence.emails[0].content);
                 setDelay(sequence.emails[0].delayInMillis);
                 setEmailId(sequence.emails[0].emailId);
-                setPublished(sequence.emails[0].published);
+                // setPublished(sequence.emails[0].published);
                 if (sequence.filter) {
                     setFilters(sequence.filter.filters);
                     setFiltersAggregator(sequence.filter.aggregator);
@@ -578,11 +578,7 @@ export default function Page({
                                         </p>
                                     </Dialog2>
                                     <Button
-                                        variant={
-                                            showScheduleInput
-                                                ? "secondary"
-                                                : "ghost"
-                                        }
+                                        variant="ghost"
                                         className="gap-2"
                                         onClick={() => {
                                             setShowScheduleInput(true);

apps/web/app/(with-contexts)/dashboard/mail/drip/[productId]/[sectionId]/page.tsx

@@ -38,7 +38,7 @@ export default function DripEmailEditorPage({
             src={`/dashboard/mail/drip/${productId}/${sectionId}/internal`}
             redirectTo={
                 redirectTo ||
-                "/dashboard/product/${productId}/content/section/${sectionId}"
+                `/dashboard/product/${productId}/content/section/${sectionId}`
             }
             title={
                 productLoaded

packages/email-editor/src/blocks/image/settings.tsx

@@ -48,8 +48,6 @@ export function ImageSettings({
     // Get numeric values from settings
     const borderRadius = pxToNumber(block.settings.borderRadius, 0);
     const borderWidth = pxToNumber(block.settings.borderWidth, 0);
-    const paddingTop = pxToNumber(block.settings.paddingTop, 0);
-    const paddingBottom = pxToNumber(block.settings.paddingBottom, 0);
 
     // Sample images
     const sampleImages = [

packages/email-editor/src/blocks/link/settings.tsx

@@ -33,11 +33,6 @@ export function LinkSettings({ block, style, updateBlock }: LinkSettingsProps) {
         return match ? Number.parseInt(match[1], 10) : defaultValue;
     };
 
-    // Get numeric values from settings
-    const fontSize = pxToNumber(block.settings.fontSize, 16);
-    const paddingTop = pxToNumber(block.settings.paddingTop, 0);
-    const paddingBottom = pxToNumber(block.settings.paddingBottom, 0);
-
     // Font size options
     const fontSizeOptions = [
         { value: "12px", label: "12px" },

packages/email-editor/src/blocks/text/settings.tsx

@@ -32,10 +32,6 @@ export function TextSettings({ block, style, updateBlock }: TextSettingsProps) {
         return match ? Number.parseInt(match[1], 10) : defaultValue;
     };
 
-    // Get numeric values from settings
-    const paddingTop = pxToNumber(block.settings.paddingTop, 0);
-    const paddingBottom = pxToNumber(block.settings.paddingBottom, 0);
-
     // Font size options
     const fontSizeOptions = [
         { value: "12px", label: "12px" },

packages/email-editor/src/components/email-editor.tsx

@@ -120,7 +120,7 @@ export function EmailEditor({
     );
     const [movingBlockId, setMovingBlockId] = useState<string | null>(null);
     const [selectedBlockId, setSelectedBlockId] = useState<string | null>(null);
-    const [showSettings, setShowSettings] = useState(true);
+    const [showSettings] = useState(true);
 
     // Update email when initialEmail prop changes
     useEffect(() => {