-
Notifications
You must be signed in to change notification settings - Fork 99
Solving issue #22, Add export to github #27
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
base: main
Are you sure you want to change the base?
Conversation
If any suggestion and feeback do let me know!!1 |
Why do we need an extra oAuth layer on this PR, since we are using github OAuth using clerk, we can simple get the access token from there and use. |
That's correct, but if a user logs in without Github, we can ask them to sign in with Github; otherwise, we can take the token from the clerk. |
Yeah, that's true. We can do that. I will try adding that also. |
github auth is not there in the current clerk which is available in the project, we might have to update the clerk setting to add github auth then we can fetch the token and use it |
yes Prashh is saying the same that if user has logged in through github to our application then we can just ask Clerk to give AuthToken to us instead of again generating Token. But if User has logged using any other method such as goggle sign in then the only option we have to authorise through github and generate github token |
#22
Screen.Recording.2025-03-12.at.9.1.mp4
https://youtu.be/L7OS7XhUbbc
Done with all the requirements, check it once,