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

Skip to content

Conversation

@pierreprinetti
Copy link
Contributor

@pierreprinetti pierreprinetti commented Oct 6, 2023

Before this patch, the documentation link generated for the Operator page resulted in a 404 Not Found on Github pages.

With this change, the documentation link points to the "Operator" anchor in the Keycloak docs index. Note that the operator docs don't seem to be versioned on the Keycloak website.

Fixes #24143

@pierreprinetti
Copy link
Contributor Author

Rebased.

Before this patch, the documentation link generated for the Operator
page resulted in a 404 Not Found on Github pages.

With this change, the documentation link points to the "Operator" anchor
in the Keycloak docs index. Note that the operator docs don't seem to be
versioned on the Keycloak website.
@pierreprinetti
Copy link
Contributor Author

Rebased.

@pierreprinetti
Copy link
Contributor Author

hmm, maybe I should ping someone. @jonkoops?

Copy link
Contributor

@jonkoops jonkoops left a comment

Choose a reason for hiding this comment

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

I am not a maintainer so I cannot merge this, but changes look good to me.

@pierreprinetti
Copy link
Contributor Author

@vmuzikar Is there anything else expected from me, before this PR can be merged?

@jonkoops
Copy link
Contributor

@pierreprinetti Please do not @ other team members, we're already pretty busy and this just adds more notifications for us. Your PR will be picked up when the maintainers have time to do so.

@jonkoops
Copy link
Contributor

Side-bar. Could you perhaps log an issue for this fix? This is something we might want to backport to the existing release, and we need an issue to track that.

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.

@pierreprinetti Thanks for the PR!

Change LGTM but we need the GH Issue as mentioned by @jonkoops. Please see https://github.com/keycloak/keycloak/blob/main/CONTRIBUTING.md

@jonkoops jonkoops requested a review from vmuzikar October 19, 2023 10:26
@jonkoops
Copy link
Contributor

@vmuzikar issue us now linked, can you take another look?

@vmuzikar vmuzikar enabled auto-merge (squash) October 19, 2023 10:35
@vmuzikar vmuzikar merged commit 2286840 into keycloak:main Oct 19, 2023
@pierreprinetti pierreprinetti deleted the operator_docs_link branch October 19, 2023 13:04
vmuzikar pushed a commit to vmuzikar/keycloak that referenced this pull request Oct 20, 2023
Before this patch, the documentation link generated for the Operator
page resulted in a 404 Not Found on Github pages.

With this change, the documentation link points to the "Operator" anchor
in the Keycloak docs index. Note that the operator docs don't seem to be
versioned on the Keycloak website.
ahus1 pushed a commit that referenced this pull request Oct 21, 2023
Before this patch, the documentation link generated for the Operator
page resulted in a 404 Not Found on Github pages.

With this change, the documentation link points to the "Operator" anchor
in the Keycloak docs index. Note that the operator docs don't seem to be
versioned on the Keycloak website.
@stianst stianst mentioned this pull request Nov 14, 2023
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.

Operator docs link is broken

3 participants