8000 Fix EntityManager API documentation link by vmuzikar · Pull Request #53759 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Fix EntityManager API documentation link - #53759

Merged
pedroigor merged 1 commit into
keycloak:mainfrom
vmuzikar:fix/53757-entitymanager-doc-link
Oct 8, 2026
Merged

pedroigor merged 1 commit into
keycloak:mainfrom
vmuzikar:fix/53757-entitymanager-doc-link

Conversation

@vmuzikar
@vmuzikar vmuzikar commented Oct 8, 2026
Copy link
Copy Markdown
Contributor

Correct the EntityManager API link in the server-development user-storage guide to use the case-sensitive EntityManager.html path instead of /entitymanager.

Closes #53757

Validation: the corrected URL returns HTTP 200.

AI disclosure: GitHub Copilot assisted with this change and PR preparation.

Co-authored-by: vmuzikar <13906855+vmuzikar@users.noreply.github.com>
Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
Closes: keycloak#53757
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vmuzikar
vmuzikar requested a review from a team as a code owner October 8, 2026 17:24
Copilot AI balanced review requested due to automatic review settings October 8, 2026 17:24
Copilot AI left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The corrected link matches the documented API path and issue requirements.

0 open findings

What changed in this PR

Corrects the Jakarta Persistence EntityManager API link to resolve the documented 404 and ExternalLinksTest failure.

Changes:

  • Uses the case-sensitive EntityManager.html API path.
File Description
docs/​documentation/​server_development/​topics/​user-storage/​entity-manager.adoc Fixes the EntityManager documentation URL.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@pedroigor
pedroigor merged commit dfdbeca into keycloak:main Oct 8, 2026
72 checks passed
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.

Broken EntityManager API link fails ExternalLinksTest

4 participants

0