-
Notifications
You must be signed in to change notification settings - Fork 136
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
Code style & docblocks #56
Conversation
Fix code style to PSR-2 & add missing docblocks
Correct minor typos in readme
Added flag for not encoded tokens + removed example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you can change that in the readme and rebase from master I can merge the PR. Thanks for the contribution.
|
||
* Regular webapp: https://auth0.com/docs/quickstart/webapp/laravel/ | ||
* Web API: https://auth0.com/docs/quickstart/backend/php-laravel/ | ||
|
||
> If you find something wrong in our docs, PR are welcome in our docs repo: https://github.com/auth0/docs | ||
|
||
### Setting up a JWKs cache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is meant to be JWK (json web keys) and not JWT. JWK is the endpoint that returns the public key to verify the token
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok will do
Fix code style to PSR-2 & add missing docblocks
Correct minor typos in readme
# Conflicts: # src/Auth0/Login/Auth0Service.php # src/Auth0/Login/LoginServiceProvider.php
@glena Please check if this PR is good to go :) |
💥 awesome! |
Fix code style to PSR-2 & add missing docblocks