-
Notifications
You must be signed in to change notification settings - Fork 146
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
feat: add Webflow provider #395
feat: add Webflow provider #395
Conversation
Hi folks at @knpuniversity 👋 Is there anything blocking this PR from being merged in its current state? |
|
||
public function getProviderClass(array $config) | ||
{ | ||
return 'Koalati\OAuth2\Client\Provider\Webflow'; |
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.
We can use return Webflow::class
here too instead of hardcoding this as a string.. but don't forget to use the namespace too :)
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.
Done! :)
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.
Hey, sorry for the delay! This looks good to me, only one minor comment
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.
Yup - I agree with Victor - just that one change should do it :)
Thanks to the both of you - and no worries on the delay, you folks do more than enough! I just changed the hardcoded string; the PHP 8.1 and 8.2 tests "failed" but the errors in these seemed to be deprecations in other parts of the codebase. Let me know if there's anything else you need me to do :) Cheers! |
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.
Thank you! It looks good me now
In order to fix the deprecation message in #395
Failed tests are really unrelated to this PR and fixed in #397 |
This PR adds an implmentation for the new
koalati/oauth2-webflow
OAuth 2.0 provider for Webflow.