-
Notifications
You must be signed in to change notification settings - Fork 433
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
Added UAEPass Provider #1203
base: master
Are you sure you want to change the base?
Added UAEPass Provider #1203
Conversation
src/UAEPass/README.md
Outdated
'uaepass' => [ | ||
'client_id' => env('UAEPASS_CLIENT_ID'), | ||
'client_secret' => env('UAEPAS_CLIENT_SECRET'), | ||
'redirect' => env('UAEPAS_REDIRECT_URI') |
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.
add env to here for enviroment
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.
Can you please elaborate ?
…nfig and other code cleanup
…nvironment variable
Co-authored-by: atymic <atymicq@gmail.com>
src/UAEPass/README.md
Outdated
'base_url' => env('UAEPASS_BASE_URL'), | ||
'client_id' => env('UAEPASS_CLIENT_ID'), | ||
'client_secret' => env('UAEPAS_CLIENT_SECRET'), | ||
'redirect' => env('UAEPAS_REDIRECT_URI') |
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.
to use $this->getConfig('base_url') you need to add base_url
here
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.
Please correct me if I'm wrong ? base_url
is already added in env as UAEPASS_BASE_URL
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.
It needs to be added to the config in services.php or getConfig
with the additional config keys wont find it
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.
Updated with default base_url in ReadMe 3caf8cb
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.
I have tested and it's working fine.
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.
one final change
This PR adds UAE Pass provider.
Official UAEPass Website