-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Tech Debt] Reorder Cases page #152989
[Tech Debt] Reorder Cases page #152989
Conversation
bbe88ab
to
a76e17a
Compare
992eda8
to
9c2c84a
Compare
8829624
to
ad8f87e
Compare
193452f
to
37b7730
Compare
a55c986
to
4982ce8
Compare
7ebbee8
to
5fc43aa
Compare
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
b3befee
to
f6fdd2e
Compare
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
Page load bundle
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
Resolves #152985
Summary
This PR updates the Cases Page to follow the newly agreed upon structure for the Observability app.
This PR is part of a larger effort to have all pages in the Observability app follow the same structure so they are more consistent with each other. More details in the epic: #152783
It also cleans up dead and unused components where found.
All design and functionality should be exactly the same as before.
Checklist
<to_name_of_page.tsx>
/container
subfolder if available/translations.ts
/app_root/components/feature
/page/<name_of_page>/components/<component_name>.tsx