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

kie-issues#2568: [sonataflow-management-console-webapp] Restyle 503 Keycloak page #2595

Merged
merged 7 commits into from
Sep 20, 2024

Conversation

kumaradityaraj
Copy link
Contributor

@kumaradityaraj kumaradityaraj commented Sep 17, 2024

Closes apache/incubator/kie-issues#2568

Description:
The KeyCloack page has been redesigned and the logo has also been changed; the UI looks smooth now.

How to test:

  • pnpm -r -F sonataflow-management-console-webapp... build:dev. run this command from the root folder
  • SONATAFLOW_MANAGEMENT_CONSOLE_WEBAPP__sonataflowEnvMode=PROD pnpm start:dev-webapp. run this command after navigating to the package/sonataflow-management-console-webapp
  • browse this link localhost:9025

Preview:
Screenshot 2024-09-17 at 1 46 56 PM

Updated Preview
Screenshot 2024-09-19 at 3 21 43 PM

@kumaradityaraj
Copy link
Contributor Author

@fantonangeli Please review this PR. Thank you.

Copy link
Contributor

@fantonangeli fantonangeli left a comment

Choose a reason for hiding this comment

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

Thanks a lot @kumaradityaraj , I left a few suggestions.

@tiagobento
Copy link
Contributor

Would it be possible to more or less follow what PatternFly itself suggests for error pages? Example: https://www.patternfly.org/2022.05/ux-writing/http-404-pages/

Copy link
Contributor

@domhanak domhanak left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@tiagobento tiagobento merged commit 2caba8d into apache:main Sep 20, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[sonataflow-management-console-webapp] Restyle 503 Keycloak page
4 participants