[ML] Kibana Management Jobs list: Ensure proper title, tagline, and link to documentation #43418
Conversation
|
Any feedback on the documentation links or particular language used is always welcome 😄 |
alvarezmelissa87
requested review from
walterra,
jgowdyelastic and
peteharverson
💚 Build Succeeded |
| // Call useRefreshInterval() after the subscription above is set up. | ||
| useRefreshInterval(setBlockRefresh); | ||
|
|
||
| if (isManagementTable === false) { |
There was a problem hiding this comment.
Hooks need to be at the top level, we cannot put them inside conditions unfortunately https://reactjs.org/docs/hooks-rules.html#only-call-hooks-at-the-top-level.
Maybe an alternative is to pass isManagementTable on to the hook and act on it there?
There was a problem hiding this comment.
That makes sense normally, but in this case isManagementTable value is never changing so it wouldn't change the order of the hooks being called. 🤔
I'm happy to change it though since maybe it does set an example for a pattern we wouldn't want to follow most of the time.
…management-updates
|
Just a quick design nit, but having the Refresh button in a row by itself creates some weird spacing issues. Since the button is tightly coupled with the data in the table, I'd suggest moving the button to be within the table's toolbar and make it a filled button. |
💚 Build Succeeded |
💔 Build Failed |
💔 Build Failed |
|
retest |
💚 Build Succeeded |
|
@cchaos - Thank you for taking a look! 😄 The We will be updating this view in subsequent iterations - it might be more appropriate then, depending on what other action buttons we add, to update the style for the |
…ink to documentation (#43418) (#43585) * Update KM jobs list title and add tag line and link to docs * add spaces column to analytics table for use in Kibana management * add refresh button to kibana management analytics jobs list * ensure refresh button does not cause bounce when loading * move refreshInterval hook to page component * add default value for analyticsTable prop
|
Pinging @elastic/ml-ui |
Summary
Follow up to #43151
Machine Learning JobsView machine learning analytics and anomaly detection jobs.Spacescolumn to analytics jobs tableChecklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.- [ ] Documentation was added for features that require explanation or tutorialsFor maintainers
- [ ] This was checked for breaking API changes and was labeled appropriately- [ ] This includes a feature addition or change that requires a release note and was labeled appropriately