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

Introduce LiveChat as oauth2 provider #1573

Closed
wants to merge 4 commits into from
Closed

Introduce LiveChat as oauth2 provider #1573

wants to merge 4 commits into from

Conversation

mariosant
Copy link
Contributor

This PR introduces LiveChat as an oauth2 provider. This will allow people to build LiveChat applications with a PocketBase backend instance.

@ganigeorgiev
Copy link
Member

ganigeorgiev commented Jan 11, 2023

Thank you. I'll review it sometime later today.

Could you add godoc comments to tools/auth/livechat.go (similar to the other providers) and if possible - link to the LiveChat users API reference?

@mariosant
Copy link
Contributor Author

mariosant commented Jan 11, 2023

Happy to do so!

@mariosant
Copy link
Contributor Author

@ganigeorgiev just updated the ui build.
Please let me know if you need help reviewing this.

ganigeorgiev added a commit that referenced this pull request Jan 12, 2023
Co-authored-by: Marios Antonoudiou <m.antonoudiou@celonis.com>
@ganigeorgiev
Copy link
Member

@mariosant I've squash merged it in the develop branch in 55c6bed
(I've applied some minor format nitpicks and added check for the email_verification field).

The provider will be available with the next v0.12.0 release.

@mariosant mariosant deleted the livechat-authentication branch January 12, 2023 20:37
abdokhaire pushed a commit to abdokhaire/postgresbase that referenced this pull request Aug 7, 2024
Co-authored-by: Marios Antonoudiou <m.antonoudiou@celonis.com>
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