Skip to content

Auth flow from pr574 - #599

Merged
blankirigaya merged 5 commits into
mainfrom
auth-flow-from-pr574
Jun 21, 2026
Merged

Auth flow from pr574#599
blankirigaya merged 5 commits into
mainfrom
auth-flow-from-pr574

Conversation

@blankirigaya

Copy link
Copy Markdown
Collaborator

Summary

auth fixes which was previously broken

Closes #


Type of Change

  • [✔️] Bug fix
  • New feature
  • Refactor (no functional change)
  • UI / Design change
  • Tests only
  • Documentation
  • Infrastructure / DevOps
  • Security

What Changed

  • auth fixed

How to Test

  1. use the auth feature to login via google or github

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev-card Ready Ready Preview, Comment Jun 19, 2026 8:56am

@github-actions github-actions Bot added backend gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. mobile web labels Jun 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi @blankirigaya,

Thanks for opening this pull request.

This PR has been automatically classified based on the files modified.

Applied Labels

  • gssoc:approved
  • backend
  • mobile
  • web

Primary Review Area

  • backend

Reviewer

@Harxhit has been identified as the primary reviewer for this pull request.

If you have any questions regarding the affected area or implementation details, feel free to reach out to the assigned reviewer.

Thank you for your contribution!

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

CI — Checks Failed

Backend — FAIL

Check Result
Lint FAIL
Test PASS
Typecheck FAIL

Mobile — SKIP

Check Result
Lint -
Test -

Web — SKIP

Check Result
Build -

Last updated: Fri, 19 Jun 2026 08:57:23 GMT

Comment thread apps/backend/prisma/migrations/20260312125106_init/migration.sql
@Harxhit

Harxhit commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

@blankirigaya Could you please remove all the migrations files from codebase and run db:migrate and push them so our migrations files can be consistent.

@Harxhit Harxhit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

After addressing requested changes and fixing merge conflicts you can merge this.

Comment thread apps/backend/src/routes/auth.ts Outdated
Signed-off-by: Pushkar Kulkarni <omplayz4456@gmail.com>
@blankirigaya blankirigaya removed web gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. labels Jun 19, 2026
@Harxhit

Harxhit commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

@blankirigaya Can you see tags

@blankirigaya
blankirigaya merged commit 9d85157 into main Jun 21, 2026
6 of 7 checks passed
Harxhit pushed a commit to Harxhit/DevCard that referenced this pull request Jun 21, 2026
* feat: apply card sharing backend changes with auth fix

* chore: update npm cleanup references

* fix: restore oauth callback validation type

* fix: restore oauth callback zod validation

---------

Signed-off-by: Pushkar Kulkarni <omplayz4456@gmail.com>
ShantKhatri pushed a commit to ShantKhatri/DevCard that referenced this pull request Jun 23, 2026
* feat: apply card sharing backend changes with auth fix

* chore: update npm cleanup references

* fix: restore oauth callback validation type

* fix: restore oauth callback zod validation

---------

Signed-off-by: Pushkar Kulkarni <omplayz4456@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants