Skip to content
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

enhancements for system logs UI #590

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aadarsh-nagrath
Copy link

Linked Issue(s)

#555 - Improvements to the system logs UI
Also deals with
#556 - Search functionality in the system logs UI

Proposed changes (including videos or screenshots)

1> Color Coding by Log Level
[INFO] (green)
[ERROR] (red)
[WARNING] (yellow)

{E7E2B491-3243-4C5D-995D-F0319E6F3FAF}

2> Highlighted Key Actions GET, POST, PUT requests could be bolded or highlighted to stand out.

3> Interactive UI with Filtering
If the logs are displayed in a UI, add interactive filters to show/hide specific log levels (e.g., only errors, warnings).
{2CA671AA-E1EE-4F83-A2BA-00A5C26CEB49}
{BB6E40EA-0CC3-479A-80AC-148C811FB6B1}

4> Search Functionality: Allow searching by timestamp, HTTP status codes, or keywords.
{15BAF3BE-23F5-4BEA-9A8F-8723E570AF63}

1> Color Coding by Log Level
[INFO]  (green)
[ERROR]  (red)
[WARNING]  (yellow)

2> Highlighted Key Actions GET, POST, PUT requests could be bolded or highlighted to stand out.

3>  Interactive UI with Filtering
If the logs are displayed in a UI, add interactive filters to show/hide specific log levels (e.g., only errors, warnings).

4> Search Functionality: Allow searching by timestamp, HTTP status codes, or keywords.
@jayantbh
Copy link
Contributor

Hey @aadarsh-nagrath, thanks for your PR. :)
I think we'll most likely go with this PR for some of UI enhancements:
#571
And this one for the search functionality:
#587

However, the log level filtering is a neat new addition not included in either of the PRs.
I'd recommend you introduce that change on top of the existing PRs (in a different PR based on those branches).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants