mirror of https://github.com/kortix-ai/suna.git
Merge pull request #1667 from KrishavRajSingh/fix/show_selected
fix: show selected schedule in quick tab
This commit is contained in:
commit
3248d1e81a
|
@ -332,7 +332,7 @@ export const SimplifiedScheduleConfig: React.FC<SimplifiedScheduleConfigProps> =
|
||||||
const [selectedMonthDays, setSelectedMonthDays] = useState<string[]>(['1']);
|
const [selectedMonthDays, setSelectedMonthDays] = useState<string[]>(['1']);
|
||||||
|
|
||||||
// One-time schedule state
|
// One-time schedule state
|
||||||
const [oneTimeDate, setOneTimeDate] = useState<Date | undefined>(new Date());
|
const [oneTimeDate, setOneTimeDate] = useState<Date | undefined>(undefined);
|
||||||
const [oneTimeHour, setOneTimeHour] = useState<string>('9');
|
const [oneTimeHour, setOneTimeHour] = useState<string>('9');
|
||||||
const [oneTimeMinute, setOneTimeMinute] = useState<string>('0');
|
const [oneTimeMinute, setOneTimeMinute] = useState<string>('0');
|
||||||
|
|
||||||
|
@ -346,11 +346,7 @@ export const SimplifiedScheduleConfig: React.FC<SimplifiedScheduleConfigProps> =
|
||||||
}, [config.cron_expression]);
|
}, [config.cron_expression]);
|
||||||
|
|
||||||
// Update cron when recurring settings change
|
// Update cron when recurring settings change
|
||||||
useEffect(() => {
|
// Removed auto-generation to prevent interference with preset selections
|
||||||
if (!selectedPreset) { // Only auto-generate if no preset is selected
|
|
||||||
handleRecurringScheduleChange();
|
|
||||||
}
|
|
||||||
}, [scheduleType, selectedHour, selectedMinute, selectedWeekdays, selectedMonthDays]);
|
|
||||||
|
|
||||||
// Update cron when one-time settings change
|
// Update cron when one-time settings change
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
|
@ -410,7 +406,10 @@ export const SimplifiedScheduleConfig: React.FC<SimplifiedScheduleConfigProps> =
|
||||||
cron_expression: cronExpression,
|
cron_expression: cronExpression,
|
||||||
timezone: timezone
|
timezone: timezone
|
||||||
});
|
});
|
||||||
setSelectedPreset(''); // Clear preset selection when using custom recurring
|
// Only clear preset if we're generating a different cron than what's currently set
|
||||||
|
if (cronExpression !== config.cron_expression) {
|
||||||
|
setSelectedPreset(''); // Clear preset selection when using custom recurring
|
||||||
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
const handleWeekdayToggle = (weekday: string) => {
|
const handleWeekdayToggle = (weekday: string) => {
|
||||||
|
|
Loading…
Reference in New Issue