Indicate the env in Sign In, Register, Forgot Password, Reset Password forms

Question

Indicate the env in Sign In, Register, Forgot Password, Reset Password forms

Closed this issue 4 months ago · 14 comments

If not in production, all of these KeyCloak forms need to indicate the environment: Sign In, Register, Forgot Password, Reset Password

Answer 1 · 2024-06-25T10:53:52.000Z

@paynejd I have tried few examples, let me know which one is better (1st one is deployed on QA)

Answer 2 · 2024-06-25T10:54:20.000Z

This value is based on KeyCloak Realm Display Name.

Answer 3 · 2024-06-25T11:06:55.000Z

@paynejd Moving it to last option, thats what I like best. Do let me know here what you think

Answer 4 · 2024-06-25T13:51:18.000Z

@paulsonder Can you take a 30sec look at this? When in a non-production environment, we just want to display which environment a user is signing into.

@snyaggarwal -- What's the business logic for determining if non-production? This should be a configuration, I assume -- the approach needs to work with many deployments (e.g OCL Online, WHO, Kenya, Brazil, etc.)

Answer 5 · 2024-06-26T03:17:40.000Z

@paynejd Each OCL environment is a separate KeyCloak Realm. A Realm has a name and a display name. Right now what you see "OCL QA" is the display name of OCL-QA Realm. Similarly "OCL Online" is the name for OCL-Prod Realm.
We don't have a configuration to tag a Realm as Prod or Non-Prod as such.

Answer 6 · 2024-06-26T10:24:04.000Z

@snyaggarwal @paynejd

How about something like this?

This uses an Alert inline, Sunny.

Answer 7 · 2024-06-26T10:37:49.000Z

@paulsonder I like the styles but I am not sure if we want to say "Server".
@paynejd Should it rather say "Environment"?

Answer 8 · 2024-06-26T11:44:57.000Z

I'll leave the label to you @snyaggarwal but updated to reflect your suggestion.

Answer 9 · 2024-06-27T04:40:13.000Z

This is deployed on QA

Answer 10 · 2024-06-27T08:22:23.000Z

@snyaggarwal The addition looks good to me.

There are a few things in the design we didn't manage to achieve in the build. I'm not sure what the limitations are, but just for clarity I've noted them below...

The language switcher is missing the icon and chevron. The design uses locale code rather than full label, but i'm comfortable with either.
The weight on the 'Sign into' is too light... it should use 'Title - Large' from Material.
The button height is smaller in code than the design
The call out for unregistered users sits in its own card in the design and includes an illustration.

Thanks!

Answer 11 · 2024-06-27T10:36:37.000Z

@paulsonder I was able to do few:

Done to very close icon.
Done
Done
Its difficult to break out the card the way the current theme is setup. Will have to move a lot of things around for this. Also I couldn't find the icon in the allowed icon list.

Answer 12 · 2024-07-01T08:20:09.000Z

@paulsonder @jamlung-ri Can this be closed?

Answer 13 · 2024-07-01T12:24:42.000Z

@paulsonder I will defer to you on this one. I took a look last week and liked what I saw.

Answer 14 · 2024-07-01T12:31:54.000Z

@snyaggarwal Yes, let's close it. I can create another ticket for point 4 when we've more time.