p2-inc / idp-wizard

Identity Provider setup wizards for Keycloak
https://phasetwo.io
Other
1 stars 1 forks source link

Branding #115

Closed xgp closed 1 year ago

xgp commented 1 year ago

Allow logo and domain/app name

Logo is in the corner domain/app name is the replacement for app.phasetwo.io.

xgp commented 1 year ago

I'll return these in the config.json:

  application data
  - domain
  - name
  - displayName
  - logoUrl

Screen Shot 2022-10-08 at 2 42 48 PM Screen Shot 2022-10-08 at 2 37 36 PM

Also requires moving any actions (IDP Selector or Logout) to the right.

xgp commented 1 year ago

Since we'll sometimes have more than one link, let's do an Application launcher in the upper left hand corner (next to the vendor logo on the wizard pages, or just standalone on the selector/dashboard pages) where the action links go.

jeffpatzer commented 1 year ago

Might be the current user I'm logged in with but current config.json content is

{
    "domain": "app.phasetwo.io",
    "name": "cloud",
    "apiMode": "cloud",
    "enableGroupMapping": true,
    "enableLdap": false,
    "enableDashboard": false,
    "emailAsUsername": false,
    "trustEmail": false
}

your comment above says it'll have and there are a two missing items

  application data
  - domain
  - name
  - displayName      # missing
  - logoUrl                # missing
xgp commented 1 year ago

Which realm are you using? cloud? wizard? It may be as simple as they're not configured. displayName comes from the realm's display name, which may not be set, and the logoUrl comes from a realm attribute, which also may not be set. If I know which realm you're using, I can update them with a value.

xgp commented 1 year ago

I added those values to cloud. https://app.phasetwo.io/auth/realms/cloud/wizard/config.json

{
  "domain": "app.phasetwo.io",
  "name": "cloud",
  "displayName": "Cloud Test",
  "logoUrl": "https://phasetwo.io/img/logo_phase_slash.svg",
  "apiMode": "cloud",
  "enableGroupMapping": true,
  "enableLdap": false,
  "enableDashboard": false,
  "emailAsUsername": false,
  "trustEmail": false
}
jeffpatzer commented 1 year ago

This was definitely it. Just missing the right info.

jeffpatzer commented 1 year ago
image image
jeffpatzer commented 1 year ago
image

without dash permission

jeffpatzer commented 1 year ago

@xgp this done except for the switch organization with hasn't been done until #114 is wrapped. Should I hide this guy? Right now its just disabled.

xgp commented 1 year ago

@jeffpatzer looks great. Let's finish both and we can merge/release at the same time.

jeffpatzer commented 1 year ago

Slight adjustment to bring the org name to the left.

image image