-
Notifications
You must be signed in to change notification settings - Fork 928
feat(site): add connection log page #18708
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
base: ethan/populate-connection-log-count
Are you sure you want to change the base?
feat(site): add connection log page #18708
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
||
return ( | ||
<Filter | ||
// TODO(ethanndickson): Add docs |
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.
Will be done in this PR stack.
f9333f1
to
5de8705
Compare
3accbf8
to
7de824c
Compare
This is ready for review, but marked as a draft so it's not merged early. |
7de824c
to
20092f6
Compare
33d8acb
to
1e3334b
Compare
20092f6
to
57acd96
Compare
1e3334b
to
332f8da
Compare
57acd96
to
24dee7a
Compare
332f8da
to
6cd9d68
Compare
5db1b73
to
4be35f4
Compare
4be35f4
to
9239251
Compare
This is the fourth PR for moving connection events out of the audit log.
This PR adds
/connectionlog
to the frontend. This page is identical in structure to the audit log, but with different filters and contents.The connection log lists sessions, and the time they start. If we support tracking the end time of a session, and we've received a disconnect event for that session, the end timestamp is also included.
Demo:
connectionlogdemo2.mov
Since the styling is identical to that of the audit log, I've continued to use MUI table components. When the audit log is migrated off MUI/restyled, this table can be too, relatively easily.
Future PRs: