fix: review comments

Author: infinitycoders05

src/components/TimezoneSelector/index.jsx

@@ -23,12 +23,11 @@ const TimezoneSelector = ({ value, onChange }) => {
   return (
     <div styleName="timezone-selector">
       <div styleName="label">Interview Timezone</div>
-      <select styleName="select" onChange={onChange}>
-        {timezoneList.map((zone, index) => (
+      <select styleName="select" onChange={onChange} value={value} defaultValue={value}>
+        {timezoneList.map((zone) => (
           <option
-            key={index}
+            key={zone}
             value={zone}
-            selected={zone === value}
           >{`${zone} - UTC${moment().tz(zone).format("Z")}`}</option>
         ))}
       </select>

src/routes/PositionDetails/components/InterviewPopup/ManageAvailability/index.jsx

@@ -124,8 +124,8 @@ const ManageAvailability = ({ scheduleDetails, onContinue }) => {
   };
 
   // Set time zone
-  const onChangeTimezone = (value) => {
-    setTimezone(value);
+  const onChangeTimezone = (event) => {
+    setTimezone(event.target.value);
   };
 
   /**
@@ -186,7 +186,7 @@ const ManageAvailability = ({ scheduleDetails, onContinue }) => {
               <div styleName="days">
                 {days.map((day, index) => (
                   <div
-                    key={index}
+                    key={day}
                     onClick={() => onDayChanged(daysMapped[index], slotIndex)}
                     styleName={cn("day", {
                       "selected-day": slot.days.indexOf(daysMapped[index]) > -1,

src/routes/PositionDetails/components/StepsIndicator/index.jsx

@@ -35,7 +35,7 @@ const StepsIndicator = ({ steps, currentStep }) => {
           const isCompleted = index <= currentStepIndex;
 
           return (
-            <div key={index}>
+            <div key={step.id}>
               <div styleName="outer-dot" style={styles} />
               <div
                 styleName={cn("dot", { "dot-completed": isCompleted })}