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
[APM] Update the sparklines to support the bar chart graph style #148702
[APM] Update the sparklines to support the bar chart graph style #148702
Conversation
Pinging @elastic/apm-ui (Team:APM) |
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.
Changes to security_solution/public/management/cypress/tsconfig.json
are good.
Hi @ashokaditya, do you know why the CI auto commit changed this file? |
💚 Build Succeeded
Metrics [docs]Async chunks
To update your PR or re-run it, just comment with: |
Yes. Sorry, I should have mentioned in the review. It's due to /pull/148496 and that we have imported from these packages within cypress tests. The fact that the second PR went after the first one (and it did not do a master sync) created the missing dependencies. |
@MiriamAparicio We should also expect the sparklines to support the comparison that we do, meaning in two colors so that way it is clear that it's taking two different periods in this case to compare. |
Hi @boriskirov, we're taking into account comparison, I think my screenshoot was not having previous data to compare, I will update it 🙏 |
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
…stic#148702) Closes elastic#146982 and elastic#125009 <img width="1068" alt="image" src="https://user-images.githubusercontent.com/31922082/211755721-2cff8d08-0b24-4f4d-b5dc-5199080094df.png"> <img width="1068" alt="image" src="https://user-images.githubusercontent.com/31922082/211808834-f58e08ba-fd35-451c-9ec0-4b4676ae1494.png"> Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Closes #146982 and #125009