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

[ui-serverLogs] Convert the Server Logs page within Administrator server in ReactJS #3927

Merged
merged 11 commits into from
Feb 4, 2025

Conversation

ananya-agarwal
Copy link
Collaborator

@ananya-agarwal ananya-agarwal commented Dec 20, 2024

What changes were proposed in this pull request?

Update the Server logs page to use React and align it with the Cloudera Design Language.
For Seeing the logs on Server Logs tab, make sure to enable Gunicorn server in local system: Document

How was this patch tested?

Unit tests, manual testing

Updated ServerLogs page:
Screenshot 2025-01-27 at 12 57 19 PM

Please review Hue Contributing Guide before opening a pull request.

@ananya-agarwal ananya-agarwal force-pushed the ServerLogs branch 2 times, most recently from 5176707 to 86d277d Compare December 24, 2024 06:16
@ananya-agarwal ananya-agarwal force-pushed the ServerLogs branch 3 times, most recently from 386b1c7 to acab827 Compare January 3, 2025 07:53
Copy link
Contributor

@JohanAhlen JohanAhlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and clean! :-)

Just some minor comments.

Copy link
Collaborator

@bjornalm bjornalm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work as usual, I have a few questions and comments before hitting approve.

In general it would be useful for the PR reviewers to see some screenshots for this since it contains new UI.

@ananya-agarwal ananya-agarwal force-pushed the ServerLogs branch 3 times, most recently from 80223b1 to 6875041 Compare January 27, 2025 11:43
@ananya-agarwal ananya-agarwal merged commit c762ef0 into master Feb 4, 2025
5 of 6 checks passed
@ananya-agarwal ananya-agarwal deleted the ServerLogs branch February 4, 2025 05:01
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.

5 participants