public/pages/Monitors/containers/Monitors/Monitors.js (6 lines): - line 45: // TODO: Abstract out a Table component to be used in both Dashboard and Monitors - line 164: // TODO: 'response.ok' is 'false' when there is no alerting config index in the cluster, and notification should not be shown to new Alerting users - line 228: // TODO: Show which values failed, succeeded, etc. - line 239: // TODO: Show which values failed, succeeded, etc. - line 277: // TODO: Show which values failed, succeeded, etc. - line 325: // TODO: Limiting to 100.. otherwise could be bringing back large amount of alerts that all need to be acknowledged 1 by 1, handle case when there are more than 100 on UI public/utils/validate.js (4 lines): - line 32: // TODO: Use a validation framework to clean all of this up or create own. - line 43: // TODO: Should clean this up later since a similar check is done in several places. - line 44: // TODO: Expand on this validation by passing in triggerValues and comparing the current - line 96: // TODO: Handle the situation that monitors with a same name can be created when user don't have the permission of 'cluster:admin/opendistro/alerting/monitor/search' public/pages/CreateTrigger/containers/CreateTrigger/CreateTrigger/CreateTrigger.js (4 lines): - line 105: // TODO: setErrors - line 143: // TODO: setErrors - line 170: // TODO: need a notification system to show errors or banners at top - line 241: // TODO: Re-enable once we have implementation to support public/pages/CreateTrigger/containers/CreateTrigger/utils/triggerToFormik.js (4 lines): - line 233: // TODO: Should move this to utils somewhere - line 250: // TODO: Removing 'params' - line 256: // TODO: Removing 'params' - line 295: // TODO: Limiting the first segment since it should not include the and/or public/pages/MonitorDetails/components/MonitorOverview/utils/getOverviewStats.js (3 lines): - line 39: // TODO: used in multiple places, move into helper - line 41: // TODO: Use OpenSearch Dashboards saved timezone (if there is one, if not THEN default to using browser) - line 65: // TODO: May be valuable to implement a toast that displays in this case. public/pages/MonitorDetails/containers/MonitorHistory/MonitorHistory.js (3 lines): - line 58: //TODO:: we might move this to DateRangePicker and make callback to update POIInterest? Check the perf and timings. - line 244: //TODO:: Handle Errors Gracefully. - line 297: //TODO:: Handle Errors Gracefully. public/pages/Destinations/containers/CreateDestination/EmailRecipients/EmailRecipients.js (3 lines): - line 45: emailGroups: [], // TODO: Possibly add emails here instead of ManageEmailGroups - line 86: // TODO: Check if 'then' is necessary here - line 91: // TODO: Only loading email groups here at the moment, should add emails as options too public/pages/Monitors/components/AcknowledgeModal/AcknowledgeModal.js (3 lines): - line 85: // TODO: Acknowledge loading, disable selection - line 86: // TODO: Empty state, no active alerts found, or too many alerts found - line 108: // TODO: Move to classname public/pages/CreateMonitor/containers/CreateMonitor/CreateMonitor.js (2 lines): - line 151: // TODO: setErrors - line 225: // TODO: setErrors public/pages/Destinations/containers/CreateDestination/ManageEmailGroups/ManageEmailGroups.js (2 lines): - line 204: // TODO: Cleanup this function (currently making sequential API calls since each one has 'awaits' on it) - line 233: // TODO: Change this to getEmailOptions from stored emailGroups state public/pages/CreateTrigger/containers/CreateTrigger/utils/formikToTrigger.js (2 lines): - line 63: // TODO: We probably also want to wrap this with 'query_level_trigger' after - line 282: // TODO: If possible, adding parentheses around the AND statements of the resulting script public/pages/Destinations/containers/DestinationsList/DestinationsList.js (2 lines): - line 168: // TODO::handle error - line 237: // TODO: 'response.ok' is 'false' when there is no alerting config index in the cluster, and notification should not be shown to new Alerting users public/pages/CreateTrigger/containers/ConfigureActions/ConfigureActions.js (2 lines): - line 151: // TODO: For bucket-level triggers, sendTestMessage will only send a test message if there is - line 245: //TODO:: Handle loading Destinations inside the Action which will be more intuitive for customers. public/pages/CreateTrigger/containers/DefineTrigger/utils/validation.js (2 lines): - line 49: // TODO: character restrictions - line 50: // TODO: character limits public/pages/CreateTrigger/containers/DefineTrigger/DefineTrigger.js (2 lines): - line 103: // TODO query-level monitor trigger graph only get the input - line 135: // TODO: need a notification system to show errors or banners at top public/pages/CreateMonitor/containers/CreateMonitor/utils/formikToMonitor.js (2 lines): - line 199: // TODO: Changing any occurrence of '.' in the fieldName to '_' since the - line 341: // TODO: Changing any occurrence of '.' in the fieldName to '_' since the public/pages/Dashboard/containers/Dashboard.js (2 lines): - line 53: // TODO: Abstract out a Table component to be used in both Dashboard and Monitors - line 218: // TODO: exists in both Dashboard and Monitors, should be moved to redux when implemented server/clusters/alerting/createAlertingCluster.js (1 line): - line 36: // TODO: Change our required header to x-
to avoid overriding public/pages/Destinations/containers/DestinationsList/utils/constants.js (1 line): - line 57: //TODO:: Convert this to proper map of text to avoid filters always cypress/integration/bucket_level_monitor_spec.js (1 line): - line 53: // TODO: Passing button props in EUI accordion was added in newer versions (31.7.0+). server/services/MonitorService.js (1 line): - line 414: //TODO: This is temporarily a pass through call which needs to be deprecated public/pages/CreateMonitor/components/MonitorType/MonitorType.js (1 line): - line 41: // TODO: Implement modal that confirms the change before clearing. public/app.js (1 line): - line 32: // TODO: review the CSS style and migrate the necessary style to SASS, as Less is not supported in OpenSearch Dashboards "new platform" anymore public/pages/CreateTrigger/containers/ConfigureTriggers/ConfigureTriggers.js (1 line): - line 101: // TODO: need a notification system to show errors or banners at top public/pages/CreateMonitor/containers/CreateMonitor/utils/whereFilters.js (1 line): - line 30: //TODO:: Breakdown to factory pattern for rules in-case we support multiple filters. This is just ease for the single one public/pages/CreateMonitor/components/MonitorExpressions/expressions/OfExpression.js (1 line): - line 37: // TODO: EuiComboBox has an internal setState issue, waiting for EUI to fix it, remove this TODO when it is fixed public/pages/MonitorDetails/containers/MonitorDetails.js (1 line): - line 157: // TODO: 404 handling public/pages/Destinations/components/createDestinations/Webhook/Webhook.js (1 line): - line 32: //TODO:: verify the Regex for all the cases based on what backend support public/pages/Destinations/containers/CreateDestination/CreateDestination.js (1 line): - line 78: // TODO:: Ideally this should fetch the destination from OpenSearch and fill in value public/pages/CreateTrigger/components/BucketLevelTriggerGraph.js (1 line): - line 66: {/*TODO: Implement VisualGraph illustrating the trigger expression similar to the implementation in TriggerGraph.js*/} public/pages/CreateMonitor/containers/DefineMonitor/DefineMonitor.js (1 line): - line 250: // TODO: Might need to check if groupBy is defined if monitor_type === Graph, and prevent onRunQuery() if no group by defined to avoid errors. public/pages/Destinations/containers/CreateDestination/EmailRecipients/utils/helpers.js (1 line): - line 38: // TODO: 'response.ok' is 'false' when there is no alerting config index in the cluster, and notification should not be shown to new Alerting users public/pages/Main/Main.js (1 line): - line 42: // TODO: Want to move this to redux store so we don't have to pass down setFlyout through components server/clusters/alerting/createAlertingADCluster.js (1 line): - line 36: // TODO: Change our required header to x-
to avoid overriding public/pages/MonitorDetails/components/MonitorHistory/TriggersTimeSeries/utils/helpers.js (1 line): - line 30: //TODO:: Confirm text with PM / UX public/pages/CreateMonitor/containers/AnomalyDetectors/AnomalyDetectors.js (1 line): - line 65: // TODO: 'response.ok' is 'false' when there is no anomaly-detection config index in the cluster, and notification should not be shown to new Anomaly-Detection users public/pages/Monitors/components/MonitorActions/MonitorActions.js (1 line): - line 46: // TODO: Support bulk acknowledge alerts across multiple monitors after figuring out the correct parameter for getAlerts API. public/pages/Destinations/containers/CreateDestination/utils/constants.js (1 line): - line 37: // TODO:: Change once we have complex forms for the URL like custom webhook public/pages/Destinations/containers/CreateDestination/ManageSenders/ManageSenders.js (1 line): - line 195: // TODO: Cleanup this function (currently making sequential API calls since each one has 'awaits' on it) public/pages/CreateTrigger/containers/CreateTrigger/utils/constants.js (1 line): - line 74: bucketSelector: JSON.stringify(FORMIK_INITIAL_BUCKET_SELECTOR_VALUES, null, 4), // TODO: To be used for Bucket-Level Triggers defined by query server/services/OpensearchService.js (1 line): - line 32: // TODO: This will be deprecated as we do not want to support accessing alerting indices directly public/pages/MonitorDetails/containers/Triggers/Triggers.js (1 line): - line 38: // TODO: For now, unwrapping all the Triggers since it's conflicting with the table public/pages/Destinations/containers/CreateDestination/utils/validations.js (1 line): - line 42: // TODO: Handle the situation that destinations with a same name can be created when user don't have the permission of 'cluster:admin/opendistro/alerting/monitor/search' public/pages/MonitorDetails/components/MonitorHistory/TriggersTimeSeries/TriggersTimeSeries.js (1 line): - line 131: {/*TODO:: Fix to always same number of intervals */} public/components/Breadcrumbs/Breadcrumbs.js (1 line): - line 140: // TODO::Everything else is considered as monitor, we should break this. public/pages/Destinations/containers/CreateDestination/EmailSender/utils/helpers.js (1 line): - line 38: // TODO: 'response.ok' is 'false' when there is no alerting config index in the cluster, and notification should not be shown to new Alerting users public/pages/Destinations/containers/CreateDestination/EmailSender/EmailSender.js (1 line): - line 72: // TODO: Check if 'then' is necessary here