Add Patreon Auth#3323
Add Patreon Auth#3323ghostdevv wants to merge 4 commits intopocketbase:masterfrom ghostdevv:patreon-auth
Conversation
|
Submitted a PR to the oauth2 package to add the patreon endpoints golang/oauth2#668 |
|
Just thought of something - it might be worth mentioning somewhere that it requires Patreon api v2 keys to be given |
Co-authored-by: GHOST <[email protected]>
|
Thank you! I've tested the implementation locally, with both creator and regular user/subscriber, and everything seems to work fine. To avoid the back-and-forth, I've squash merged it directly in the If the Go team ended up merging the endpoints PR we can update it later, but even with the static endpoints is fine (at the end the The change will be available with the next PocketBase v0.19.0 release, but please note that there are no fixed ETAs yet (see also #3271). If you want to generate a static binary from the
|
|
Ty! |
Co-authored-by: GHOST <[email protected]>
Adds the patreon endpoints from their documentation. https://docs.patreon.com/#oauth. I've tested these thoroughly whilst making a pr to pocketbase. pocketbase/pocketbase#3323. Change-Id: I023871a4340ee4ebcba9fd808d4a8f0c081fc10e GitHub-Last-Rev: 890ef05 GitHub-Pull-Request: #668 Reviewed-on: https://go-review.googlesource.com/c/oauth2/+/528640 Reviewed-by: Cherry Mui <[email protected]> Reviewed-by: Carlos Amedee <[email protected]> Reviewed-by: Jorropo <[email protected]> Auto-Submit: Jorropo <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]>
I based this off the Yandex commit 435eca6#diff-0c7bbc200a025d45ef4358425b0a3c41b8de1e8b9d5f4b94a09f1d5c612e4481 - let me know if there is anything missing and I will add it to this pr! I tested this auth with the pocketbase js sdk and also running
make testandmake lint