Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@wimdeblauwe
Copy link
Contributor

@wimdeblauwe wimdeblauwe commented Jun 13, 2024

Fix the website on https://www.keycloak.org/server/db showing some asciidoc related ifeval text

Closes #30417

@mhajas
Copy link
Contributor

mhajas commented Jun 13, 2024

@wimdeblauwe Thank you for the fix! Could you please signoff your commit? https://github.com/keycloak/keycloak/pull/30394/checks?check_run_id=26165830605

@ahus1
Copy link
Contributor

ahus1 commented Jun 13, 2024

@wimdeblauwe did you try to render it locally? As ifeval is a block preprocessor element it needs to be on the start of a new line, so we might be missing a newline instead?

stianst
stianst previously approved these changes Jun 13, 2024
@wimdeblauwe
Copy link
Contributor Author

I did not render it locally. I have never build Keycloak or the documentation. I just saw the wrong text on the website and thought I might be able to create my smallest PR ever :) Would be good if somebody could validate that it really fixes the problem.

@vmuzikar
Copy link
Contributor

I did build it locally and can confirm the problem still persists, as hinted by @ahus1.

wimdeblauwe and others added 2 commits June 13, 2024 21:45
Fix the website on https://www.keycloak.org/server/db showing some asciidoc related `ifeval` text

Closes keycloak#30417

Signed-off-by: Wim Deblauwe <[email protected]>
@ahus1
Copy link
Contributor

ahus1 commented Jun 13, 2024

I've pushed another commit, and also linked this to an issue so we can better backport it to KC25.

See below a screenshot of the newly rendered text.

image

@ahus1 ahus1 self-assigned this Jun 13, 2024
@ahus1 ahus1 requested a review from vmuzikar June 13, 2024 20:03
@ahus1
Copy link
Contributor

ahus1 commented Jun 13, 2024

@vmuzikar - please review when you have the time. Thanks!

Copy link
Contributor

@vmuzikar vmuzikar left a comment

Choose a reason for hiding this comment

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

@ahus1 @wimdeblauwe LGTM, thank you.

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.

Keycloak 25 db guide shows unevaluated "ifeval

5 participants