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
Google auth website #2391
Google auth website #2391
Conversation
2006e7c
to
c115ccd
Compare
3b895cd
to
fdfa345
Compare
@@ -51,7 +52,8 @@ const ManageUser = ({ user }: InferGetServerSidePropsType<typeof getServerSidePr | |||
const toast = useToast(); | |||
|
|||
const { data: stats } = uswSWRImmutable<Partial<{ [time in LeaderboardTimeFrame]: LeaderboardEntity }>>( | |||
"/api/user_stats?uid=" + user.id, | |||
"/api/user_stats?" + | |||
new URLSearchParams({ id: user.id, auth_method: user.auth_method, display_name: user.display_name }), |
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.
fix a bug we already have now:
if you go to the prod deployment / admin / open a user that is logged in with discord, the stats are missing at the bottom.
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.
Notice that as well, but I'm too lazy to fix it lol
Add google login to website
lucide Icons has no Google icon, and as it turns out, they don't allow any new branch icons lucide-icons/lucide#670 and will be removing old ones.
So only for logos of brands, we can use this new library, otherwise we stick to lucide