Skip to content
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 User Profile #66

Merged
merged 8 commits into from
Jun 17, 2024
Merged

Add User Profile #66

merged 8 commits into from
Jun 17, 2024

Conversation

mrti259
Copy link
Member

@mrti259 mrti259 commented Jun 15, 2024

@gabokatta
Copy link
Contributor

Comento aca para cuando la mergees me notifique :D

@mrti259
Copy link
Member Author

mrti259 commented Jun 16, 2024

Comento aca para cuando la mergees me notifique :D

ya lo encare yo #ansioso

@gabokatta
Copy link
Contributor

Comento aca para cuando la mergees me notifique :D

ya lo encare yo #ansioso

Ohhh oki oki, yo la habia arrancado ayer pero dale, me concentro en la otra entonces, asi no te piso

@mrti259
Copy link
Member Author

mrti259 commented Jun 16, 2024

Merge after #67 is merge with this branch

Depends: VaquitApp/backend#66

mrti259 and others added 2 commits June 16, 2024 16:34
Co-authored-by: Tomás Grüner <47506558+MegaRedHand@users.noreply.github.com>
@mrti259 mrti259 marked this pull request as ready for review June 16, 2024 19:52
VITE_API_URL="http://localhost:8000"

PUBLIC_GOOGLE_CLIENT_ID="99613318645-jd627o7b5ot8p78dfpeg0duandg766p5.apps.googleusercontent.com"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

esto esta OK que quede configurado?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

si, es publico. si inspeccionas el html tambien se lo ve. hay uno secret que no se usa
lo deje para que puedan probarlo. igual lo tienen que agregar al .env

@mrti259 mrti259 merged commit 65c0c61 into main Jun 17, 2024
1 check passed
@mrti259 mrti259 deleted the profile branch June 17, 2024 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Datos Bancarios [3]
2 participants