next-auth
next-auth copied to clipboard
nextauthjs
Add Tencent QQ Provider
Reasoning 💡
Add Tencent QQ Provider I'm a beginner and I'm providing the provider I've used here, if there are any mistakes, please point them out.
Checklist 🧢
- [x] Documenta0tion
- [ ] Tests
- [x] Ready to be merged
I define and use it directly in the [...nextauth].js file, everything works fine, I don't know if this counts as a pass test (because the function name magically adds the Provider suffix in the merged project).
Affected issues 🎟
This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: https://vercel.com/nextauthjs/next-auth/82fzcCmqm8Y4PrJjCMW6Cryno3fa
✅ Preview: https://next-auth-git-fork-tian-que-main-nextauthjs.vercel.app
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
Great, thanks! In general, looks good to me. We're currenlty on a break merging new features (like new providers). So I've labelled this PR as "ready" so the other maintainers can easily find it when everyone is back!
It looks like this issue did not receive any activity for 60 days. It will be closed in 7 days if no further activity occurs. If you think your issue is still relevant, commenting will keep it open. Thanks!
![stale[bot] avatar](https://avatars.githubusercontent.com/in/1724?v=4 "Published by a pub.dev verified publisher"){kind=small}
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated |
|---|---|---|---|
| next-auth | ✅ Ready (Inspect) | Visit Preview | Jul 10, 2022 at 6:55PM (UTC) |
To keep things tidy, we are closing this issue for now. If you think your issue is still relevant, leave a comment and we might reopen it. Thanks!
@Tian-que yes, as mentioned above. We are striving to have all our providers written in typescript.
Are you still interested in getting this provider merged? Can you make the TS refactor?
@Tian-que yes, as mentioned above. We are striving to have all our providers written in typescript.
Are you still interested in getting this provider merged? Can you make the TS refactor?
Sorry I don't have enough time to do it right now (to be honest, I've forgotten how to use TypeScript), please close first. wait till i have time to do it.
@Tian-que yes, as mentioned above. We are striving to have all our providers written in typescript. Are you still interested in getting this provider merged? Can you make the TS refactor?
Sorry I don't have enough time to do it right now (to be honest, I've forgotten how to use TypeScript), please close first. wait till i have time to do it.
Okay no worries, thanks for being open and honest! I'll leave it as is for now. The bot will close it automatically after a while if no one else is interested and comes along.
