Skip to content

Conversation

@longshuicy
Copy link
Member

To test:

  1. run the dev script sh docker-dev.sh up
  2. login to administrator console, find theme
  3. select clowder-theme and save

image

go to clowder frontend and try login, you should see an interface like this:
image

I followed tutorial of https://medium.com/keycloak/change-login-theme-in-keycloak-docker-image-55b5fa5ceec4
CSS is adjusted based on this.

@longshuicy longshuicy requested a review from lmarini May 12, 2022 19:38
Base automatically changed from Registration-with-Keycloak-#3 to main May 13, 2022 20:34
if [ $1 == "up" ]
then
docker-compose -f docker-compose.dev.yml up -d
# need to add volume so we can copy the clowder theme into the right place inside the container
Copy link
Member

Choose a reason for hiding this comment

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

why not just add the volume to the docker compose? with this approach if anyone runs the default docker compose to run the full stack it will not include this.

"resetPasswordAllowed": false,
"editUsernameAllowed": false,
"resetPasswordAllowed": true,
"editUsernameAllowed": true,
Copy link
Member

Choose a reason for hiding this comment

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

I am not sure we want to allow them to change their username? I think we just want the username to default to their email address?

lmarini added 2 commits June 8, 2022 16:30
# Conflicts:
#	backend/keycloak/clowder-realm-dev.json
Removed healthcheck for keycloak.
@lmarini lmarini merged commit 052bdc0 into main Jun 22, 2022
@lmarini lmarini deleted the set-keycloak-login-theme branch June 22, 2022 14:52
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.

3 participants