From 754de3be4f1f09e5868384c6dee76d3428159c2d Mon Sep 17 00:00:00 2001 From: Jedr Blaszyk Date: Fri, 26 Jul 2024 14:04:03 +0200 Subject: [PATCH] [Search/Connector] Make minute-level scheduling default option in ConnectorCronEditor (#189250) ## Summary Enable minute level scheduling in UI for all connector sync jobs. This fixes the bug with minute level scheduling not present for incremental sync jobs. It seems that during [this migration](https://github.com/elastic/kibana/commit/bb5ca2e187c6c302625c38ccd85ae1965c47fdbc#diff-328630e75e33bdb823bad243330681942ab5a391c5b83cad7f3186c4ca651e9dL219) the minute level scheduling for incremental syncs was lost somehow. As of now the `ConnectorCronEditor` is only used for connector sync jobs, all connector sync jobs support minute-level scheduling so I deleted the `MINUTE` from default `frequencyBlockList` in the component. Screenshot 2024-07-26 at 10 32 16 --- .../components/scheduling/connector_cron_editor.tsx | 2 +- .../components/scheduling/full_content.tsx | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/packages/kbn-search-connectors/components/scheduling/connector_cron_editor.tsx b/packages/kbn-search-connectors/components/scheduling/connector_cron_editor.tsx index 629f2b432fd11..d5c8572dfa595 100644 --- a/packages/kbn-search-connectors/components/scheduling/connector_cron_editor.tsx +++ b/packages/kbn-search-connectors/components/scheduling/connector_cron_editor.tsx @@ -26,7 +26,7 @@ interface ConnectorCronEditorProps { export const ConnectorCronEditor: React.FC = ({ dataTelemetryIdPrefix, disabled = false, - frequencyBlockList = ['MINUTE'], + frequencyBlockList = [], hasSyncTypeChanges, onReset, onSave, diff --git a/packages/kbn-search-connectors/components/scheduling/full_content.tsx b/packages/kbn-search-connectors/components/scheduling/full_content.tsx index 3a453605f82e2..76051923ff033 100644 --- a/packages/kbn-search-connectors/components/scheduling/full_content.tsx +++ b/packages/kbn-search-connectors/components/scheduling/full_content.tsx @@ -216,9 +216,6 @@ export const ConnectorContentScheduling: React.FC { setScheduling({