fix: don't update schedule if frequency is alread set to the same value (NangoHQ#2380)

We were always updating the frequency even when we didn't need to.

This doesn't address the log saying frequency has been updated. I will
do that later

## Checklist before requesting a review (skip if just adding/editing
APIs & templates)
- [ ] I added tests, otherwise the reason is: 
- [ ] I added observability, otherwise the reason is:
- [ ] I added analytics, otherwise the reason is:

Author: TBonnin

packages/scheduler/lib/scheduler.ts

@@ -383,13 +383,17 @@ export class Scheduler {
      */
     public async setScheduleFrequency({ scheduleName, frequencyMs }: { scheduleName: string; frequencyMs: number }): Promise<Result<Schedule>> {
         return this.dbClient.db.transaction(async (trx) => {
-            const schedule = await schedules.search(trx, { names: [scheduleName], limit: 1, forUpdate: true });
+            const schedule = await schedules.search(trx, { names: [scheduleName], limit: 1 });
             if (schedule.isErr()) {
                 return Err(schedule.error);
             }
             if (!schedule.value[0]) {
                 return Err(`Schedule '${scheduleName}' not found`);
             }
+            // No-op if the schedule is already in the desired frequency
+            if (schedule.value[0].frequencyMs === frequencyMs) {
+                return Ok(schedule.value[0]);
+            }
             const res = await schedules.update(trx, { id: schedule.value[0].id, frequencyMs });
             if (res.isErr()) {
                 return Err(`Error updating schedule frequency '${scheduleName}': ${stringifyError(res.error)}`);

packages/shared/lib/services/sync/manager.service.ts

@@ -131,7 +131,7 @@ export class SyncManagerService {
                 }
 
                 const createdSync = await createSync(connection.id as number, syncName);
-                orchestrator.scheduleSyncHelper(
+                await orchestrator.scheduleSyncHelper(
                     connection,
                     createdSync as Sync,
                     syncConfig as ProviderConfig,