Skip to content

fix: Username exists when creating new user - #13

Merged
ItsAbhinavM merged 2 commits into
amfoss:mainfrom
Nishthajain7:username-taken
Apr 18, 2026
Merged

fix: Username exists when creating new user#13
ItsAbhinavM merged 2 commits into
amfoss:mainfrom
Nishthajain7:username-taken

Conversation

@Nishthajain7

Copy link
Copy Markdown
Contributor

Related Issues

Fixes #11

Description

When a new user registers using OAuth, they are redirected to complete their profile and set a username. This PR checks if a username already exists in the db.

@ItsAbhinavM
ItsAbhinavM self-requested a review April 15, 2026 16:58

@ItsAbhinavM ItsAbhinavM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The current work looks good. I see that for auth there are functions like login for existing users and google_verify for new users. How is login for google auth users handled, is there any endpoint for that ?

  • Also a suggestion that the function name complete_profile seems a bit confusing if you are registering new users in that function. Renaming it to google_register would be good.

@ItsAbhinavM ItsAbhinavM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ItsAbhinavM
ItsAbhinavM merged commit c7b13b3 into amfoss:main Apr 18, 2026
1 check passed
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.

Handle "Username Exists" when creating a new user

2 participants