Skip to content

Register User - API Integration#27

Merged
archa8 merged 4 commits into
mainfrom
avdhesh/client/register-user
Aug 17, 2025
Merged

Register User - API Integration#27
archa8 merged 4 commits into
mainfrom
avdhesh/client/register-user

Conversation

@Avdhesh-Varshney
Copy link
Copy Markdown
Collaborator

@Avdhesh-Varshney Avdhesh-Varshney commented Aug 17, 2025

Closes #26

  • Update all the routing setup & login flow from main.tsx to app.tsx file
  • Create register user api function to integrate the FE with BE

API call on login

image

@Avdhesh-Varshney Avdhesh-Varshney self-assigned this Aug 17, 2025
Copilot AI review requested due to automatic review settings August 17, 2025 12:49
@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 17, 2025

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

Project Deployment Preview Comments Updated (UTC)
password-manager Ready Ready Preview Comment Aug 17, 2025 0:59am

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR restructures the authentication flow by moving routing logic from main.tsx to App.tsx and implements user registration functionality to integrate the frontend with the backend.

  • Moved authentication routing logic from main.tsx to App.tsx for better component organization
  • Added user registration API integration that automatically registers users upon sign-in
  • Created TypeScript interfaces and API function to support user registration workflow

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
client/src/typings/index.ts Defines TypeScript interfaces for user registration payload and response
client/src/requests/user.ts Implements registerUser API function to communicate with backend
client/src/main.tsx Removes authentication routing logic and imports, simplifying the main entry point
client/src/App.tsx Adds authentication routing logic and user registration side effect
client/package.json Adds axios dependency for HTTP requests
client/.env.example Adds server domain environment variable configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread client/src/requests/user.ts Outdated
Comment thread client/src/App.tsx
Comment thread client/src/App.tsx Outdated
Comment thread client/src/App.tsx
@Avdhesh-Varshney Avdhesh-Varshney changed the title Avdhesh/client/register user Register User - API Integration Aug 17, 2025
Copy link
Copy Markdown
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

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

App.tsx and main.tsx look cleaner now. LGTM!

@archa8 archa8 merged commit 81e2760 into main Aug 17, 2025
2 checks passed
@archa8 archa8 deleted the avdhesh/client/register-user branch August 17, 2025 16:06
@Avdhesh-Varshney
Copy link
Copy Markdown
Collaborator Author

App.tsx and main.tsx look cleaner now. LGTM!

Thanks! 😃

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.

Register User API integration

3 participants