Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Rename 'last updated time' to 'last enabled time' on Detector list #233

Conversation

yizheliu-amazon
Copy link
Contributor

Issue #, if available:

Description of changes:
Rename 'last updated time' to 'last enabled time' on Detector list

Screen Shot 2020-06-22 at 10 52 29 AM

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@yizheliu-amazon yizheliu-amazon added the enhancement Enhance current feature for better performance, user experience, etc label Jun 22, 2020
detectorsWithJobResponses: any[]
): any[] => {
const finalDetectorsWithJobResponses = cloneDeep(detectorsWithJobResponses);
const resultDetetorWithJobs = [] as any[];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small: typo in Detector

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

Copy link
Contributor

@ohltyler ohltyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Left one small comment

@yizheliu-amazon yizheliu-amazon merged commit ef9c6e2 into opendistro-for-elasticsearch:master Jun 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Enhance current feature for better performance, user experience, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants