-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add folder download to frontend #32
Conversation
✅ Deploy Preview for auto-drive-demo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Solves: #5 |
frontend/src/app/page.tsx
Outdated
@@ -16,7 +16,7 @@ export default function App() { | |||
// These are example parameters. Replace with your actual values | |||
const params = new URLSearchParams({ | |||
client_id: process.env.NEXT_PUBLIC_GOOGLE_AUTH_CLIENT_ID!, | |||
redirect_uri: "http://localhost:8080/api/auth/callback/google", | |||
redirect_uri: `${window.location.origin}/api/auth/callback/google`, |
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.
I don't want to make it a blocker, if this work, this is fine with me.
But since next-auth is setup, why not simply using signIn('google') from next-auth when clicking on the google signin button?
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.
Right, much cleaner solution
Added folder download to frontend and couple small fixes