-
Notifications
You must be signed in to change notification settings - Fork 8.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Upgrade Assistant] Move fix deprecation logs into the es_deprecation_logs page #118688
[Upgrade Assistant] Move fix deprecation logs into the es_deprecation_logs page #118688
Conversation
} | ||
// Depending upon setHasNoDeprecationLogs would create an infinite loop. | ||
// eslint-disable-next-line react-hooks/exhaustive-deps | ||
}, [error, isLoading, hasLogs]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was part of the logic used used for setting the step completion when there was no deprecation logs.
hasPrivileges, | ||
privilegesMissing, | ||
}) => { | ||
const FixLogsStep: FunctionComponent<Props> = ({ hasPrivileges, privilegesMissing }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: We should rename this function to something like FixDeprecationLogs
, and the same goes for the file itself
|
||
// Depending upon setIsComplete would create an infinite loop. | ||
// eslint-disable-next-line react-hooks/exhaustive-deps | ||
}, [isDeprecationLogIndexingEnabled]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was part of the logic used used for setting the step completion when there was no deprecation logs.
💔 Build FailedFailed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
Page load bundle
HistoryTo update your PR or re-run it, just comment with: cc @sabarasaba |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I will do the rename of the fix deprecation log on my branch. Cheers! 👍
Removes step completion logic and moves
fix_deprecation_logs
intoes_deprecations_logs
folder.Tests are expected to fail, but will be addressed in a subsequent PR.