-
Notifications
You must be signed in to change notification settings - Fork 6
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
Oauth updates #28
base: master
Are you sure you want to change the base?
Oauth updates #28
Conversation
Co-authored-by: William <sirwilliamthefirst@users.noreply.github.com>
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.
Thanks for starting work on the v0.20.x changes! I just made a quick comment for how to approach any newly added values in the API. Let me know if you need further guidance on adding/removing fields.
I just committed a couple of things:
Aside from that, I believe this looks good to me! |
There are a good amount of updates to the lemmy v0.20.0 API. This PR fixes the ones that are required to run with v0.20.0 and to support OAuth/SSO. There may be additional api changes also.
LemmyNet/lemmy@0.19.6...main
LemmyNet/lemmy-js-client@0.19.6...0.20.0-alpha.18
Some v0.20.0 api changes that I see so far:
I am following this blog post from @privacyguard to implement the SSO
https://privacyportal.org/blog/sign-in-with-privacy-portal-tutorial