-
Notifications
You must be signed in to change notification settings - Fork 6
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
Improve documentation on configuring plugin #93
Comments
+1 on this. I personally got stuck when defining the clients. I have no idea where to find the necessary info to create a new client (in my case, I would like to use my site to log in to OpenStreetMap). |
+1. I Agree. It's pretty hard guessing out of the blue what strings should be changed to what when registering clients in the current state of the README. That makes a lot of pretty unclear parameters at once to play with. Just a little context or examples would be very useful here. |
Docs are not too clear on how to configure the keys. We should mention specifically that they should be set in
wp-config.php
, and could also mention usingwp-cli
to set it.Docs are also not too clear on how to set
oidc_clients
. It's suggested to usefunctions.php
, which is not ideal because it's tied to the theme. We should probably suggest using a plugin (or probably even an mu-plugin) to configure theoidc_clients
.The text was updated successfully, but these errors were encountered: