Skip to content
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

Update documentation to use symfony flex recipe #113

Merged
merged 6 commits into from
Sep 21, 2018

Conversation

ckrack
Copy link
Contributor

@ckrack ckrack commented Aug 21, 2018

Update docs to refer to the symfony recipe-contrib.
Update references to yaml configs.
Use env vars instead of parameters.yml.
Update interfaces for autowiring.

Requested in #112

ckrack and others added 3 commits August 21, 2018 23:21
Update docs to refer to the symfony recipe-contrib.
Update references to yaml configs.
Use env vars instead of parameters.yml.
Update interfaces for autowiring.
@weaverryan
Copy link
Member

Hey @ckrack!

You did an awesome job! Thank you! I proofread it an made some small tweaks. Can you tell me if everything looks ok to you?

Also, the entire Configuration section is actually automatically updated via this (ugly) script: https://github.com/knpuniversity/oauth2-client-bundle/blob/master/bin/update_readme

Would you be interested in updating that script so that it generates the changes you made to the Configuration section (all the %env(...%)) changes?

Cheers!

@weaverryan
Copy link
Member

Ping @ckrack! :)

Use env vars (uppercase).
The Authenticatorinterface requires us to implement three methods,
that were previously undocumented in the example.

Fixes knpuniversity#108
@ckrack
Copy link
Contributor Author

ckrack commented Sep 11, 2018

This should also fix #101 and #108.

FlintCI seems to be lacking some configuration.

@weaverryan weaverryan merged commit f3ba077 into knpuniversity:master Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants