-
Notifications
You must be signed in to change notification settings - Fork 99
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
🐛 FIX(github/email): fix github oauth scope & callback handler #201
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @moinulmoin on Vercel. @moinulmoin first needs to authorize it. |
hey @moinulmoin |
If you hadn't mixed it with irrelevant changes, I would have been happy to merge. Thanks for the PR though. |
@moinulmoin what was irrelevant ?
simply we can rename this PR if you think the name was more clear |
okay, please rename this pr and add the following changes, then we are good to merge. |
@moinulmoin is this better ? I also rebased the branch so it doesn't conflict with latest merges and keeps the commits clean |
Awesome bro, good job. Thank you so much for the PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
All thanks to you! |
I believe this was missed