8000 Fix Keycloak Integration Documentation by lkiesow · Pull Request #6107 · hedgedoc/hedgedoc · GitHub
[go: up one dir, main page]

Skip to content

Fix Keycloak Integration Documentation - #6107

Merged
ErikMichelson merged 2 commits into
hedgedoc:masterfrom
lkiesow:hd1-keycloak-docs
Aug 14, 2025
Merged

ErikMichelson merged 2 commits into
hedgedoc:masterfrom
lkiesow:hd1-keycloak-docs

Conversation

@lkiesow
@lkiesow lkiesow commented Aug 7, 2025 •
Copy link
Copy Markdown

Component/Part

  • Documentation
  • HedgeDoc 1.x

Description

Several URLs patterns in the Keycloak guide for HedgeDoc 1 are (have become?) incorrect. For users, that makes the integration harder than it needs to be.

This patch updates the documentation page to reflect the current state of Keycloak.

Steps

  • Added implementation
  • Added / updated tests
  • Added / updated documentation
  • Added changelog snippet
  • I read the contribution documentation and
    made sure that:
    • My commits are signed-off to accept the DCO.
    • This PR targets the correct branch: master for 1.x & docs, develop for 2.x

Preview

Screenshot from 2025-08-07 15-34-13

Several URLs patterns in the Keycloak guide for HedgeDoc 1 are (have
become?) incorrect. For users, that makes the integration harder than it
needs to be.

This patch updates the documentation page to reflect the current state
of Keycloak.

Signed-off-by: Lars Kiesow <lkiesow@uos.de>
@lkiesow
lkiesow force-pushed the hd1-keycloak-docs branch from 560d4fb to 419b173 Compare August 7, 2025 13:52
@ErikMichelson ErikMichelson left a comment
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you very much for updating this guide! Looks good, just a little typo.

Comment thread docs/content/guides/auth/keycloak.md Resolved
Co-authored-by: Erik Michelson <github@erik.michelson.eu>
Signed-off-by: Lars Kiesow <lkiesow@uos.de>
@lkiesow
lkiesow commented Aug 14, 2025
Copy link
Copy Markdown
Author

Sorry about that. Applied your suggestion

@ErikMichelson ErikMichelson left a comment
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perfect, thanks!

@ErikMichelson
ErikMichelson enabled auto-merge (rebase) August 14, 2025 12:43
@ErikMichelson
ErikMichelson disabled auto-merge August 14, 2025 12:43
@ErikMichelson
ErikMichelson enabled auto-merge (squash) August 14, 2025 12:43
@ErikMichelson
ErikMichelson merged commit eba0fc5 into hedgedoc:master Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants

0