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

Setting up Keycloak che-username-readonly-theme #1723

Merged
merged 9 commits into from
Dec 10, 2020
Merged

Setting up Keycloak che-username-readonly-theme #1723

merged 9 commits into from
Dec 10, 2020

Conversation

tolusha
Copy link
Contributor

@tolusha tolusha commented Dec 9, 2020

Signed-off-by: Anatolii Bazko [email protected]

Read our Contribution guide before submitting a PR.

What does this PR do?

Add section to set up Keycloak che-username-readonly-theme

What issues does this PR fix or reference?

https://issues.redhat.com/browse/RHDEVDOCS-2453

PR Checklist

As the author of this Pull Request I made sure that:

  • vale has been run successfully against the PR branch
  • Link checker has been run successfully against the PR branch
  • Documentation describes a scenario that is already covered by QE tests, otherwise an issue has been created and acknowledged by Che QE team
  • Changed article references are updated where they are used (or a redirect has been set up on the docs side):

screenshot-0 0 0 0_4000-2020 12 09-11_47_16

@tolusha tolusha requested review from sleshchenko and removed request for a team December 9, 2020 09:48
@tolusha tolusha mentioned this pull request Dec 9, 2020
56 tasks
@MichalMaler
Copy link
Contributor

Hello Anatolii. Nice work. Do you think you could send me the equivalent of those screenshots from CRW? Since there is the CHE branding, it will appear in the CRW docs after downstream. I will fix that but need those crw screenshots.

@tolusha
Copy link
Contributor Author

tolusha commented Dec 9, 2020

It is committed in the PR
installation-guide/images/keycloak/keycloak_codeready_username_readonly_theme.png

Signed-off-by: Anatolii Bazko <[email protected]>
Signed-off-by: Anatolii Bazko <[email protected]>
@MichalMaler
Copy link
Contributor

Good job with those screenshots. Let me check it now.

@MichalMaler MichalMaler changed the title Setting up Keycloak che-username-readonly-theme WIP:Setting up Keycloak che-username-readonly-theme Dec 9, 2020
Copy link
Contributor

@MichalMaler MichalMaler left a comment

Choose a reason for hiding this comment

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

Need to fix the heading and create some intro.

@MichalMaler MichalMaler self-requested a review December 9, 2020 12:17
MichalMaler and others added 6 commits December 9, 2020 14:39
Signed-off-by: Michal Maléř <[email protected]>
Co-authored-by: Robert Krátký <[email protected]>
Signed-off-by: Michal Maléř <[email protected]>
Signed-off-by: Michal Maléř <[email protected]>
Signed-off-by: Michal Maléř <[email protected]>
@MichalMaler MichalMaler changed the title WIP:Setting up Keycloak che-username-readonly-theme Setting up Keycloak che-username-readonly-theme Dec 10, 2020
@MichalMaler MichalMaler merged commit 2a7b0a3 into eclipse-che:master Dec 10, 2020
yhontyk pushed a commit that referenced this pull request Dec 11, 2020
* Setting up Keycloak che-username-readonly-theme

Signed-off-by: Anatolii Bazko <[email protected]>

* Fixes

Signed-off-by: Anatolii Bazko <[email protected]>

* Fixes

Signed-off-by: Anatolii Bazko <[email protected]>

* Minor changes

Signed-off-by: Michal Maléř <[email protected]>

* Apply suggestions from code review

Co-authored-by: Robert Krátký <[email protected]>

* renaming file to make the match

Signed-off-by: Michal Maléř <[email protected]>

* Edits

Signed-off-by: Michal Maléř <[email protected]>

* removing unused attribute

Signed-off-by: Michal Maléř <[email protected]>

Co-authored-by: Michal Maléř <[email protected]>
Co-authored-by: Michal Maléř <[email protected]>
Co-authored-by: Robert Krátký <[email protected]>
@yhontyk yhontyk added cherry-picked to 7.32.x cherry-picked to 7.32.x and removed need-cherry-pick/7.32.x need cherry-pick to 7.32.x labels Dec 11, 2020
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.

4 participants