-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix: removed outdated information in doc configuration section #2723
Conversation
Thank you for these changes! Unfortunately almost all documents - except the Dockerfile - are autogenerated and need to fixed elsewhere. Here are some of those files:
Also keep in mind that the $HOME behavior was changed only recently. I don’t recall which version exactly but it is the one where we introduced Koanf / the new config handling! |
This reverts commit a959e75.
Thanks for the clarification 👍🏻 It seems that koanf was introduced with ory hydra v1.9.0 release That would mean that we need to maintain two versions of docusaurus-templates:
|
Great, thank you! Would you also be open to make the change to ? :) |
Ah, nvm, I quickly removed it myself :) |
Related issue(s)
Checklist
contributing code guidelines.
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
Further Comments
This fixes #2452