-
Notifications
You must be signed in to change notification settings - Fork 118
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
Line Login v2.1 update request which I've done #172
Comments
Maybe @alexfilatov would prefer to move the implementation under Ueberauth organization 🤷🏻 |
Happy to do so @yordis |
@MTheProgrammer sorry, looks like I missed your PR completely, just merged, thank you! 🙏 Just looked at your fork, really good job! You can make a PR against my repo and after that I will move it under the ueberauth org. |
@alexfilatov Thank you, I had to rewrite the library because Line Login API v2.1 does not return necessary user details. You have to request it from another endpoint. Here are docs about PKCE compatibility: |
Problem Statement
Hi, I've created an updated ingegration for Line Login that supports their new API v2.1. Current integration does not work and the maintainer is not responding.
Shall I add this as a new integration or replace the old url? Should it added to hex under my namespace?
It needs some polishing yet, however I should be done with it soon:
https://github.com/LucidModules/ueberauth_line
Solution Brainstorm
No response
The text was updated successfully, but these errors were encountered: