Skip to content

Add Register page with routing and navigation button#24

Open
Bala327 wants to merge 3 commits intoAyush0316:mainfrom
Bala327:feature/register-page
Open

Add Register page with routing and navigation button#24
Bala327 wants to merge 3 commits intoAyush0316:mainfrom
Bala327:feature/register-page

Conversation

@Bala327
Copy link

@Bala327 Bala327 commented Jul 24, 2025

Creation of Register page[frontend]

@Ayush0316
Copy link
Owner

  1. Always include screenshots whenever you work on any UI-related task.
  2. As I already mentioned on the Discord server, if you need to use a button, use the Button component located at src/components/common/buttons.

@Bala327
Copy link
Author

Bala327 commented Jul 24, 2025

sir here is my screenshot of my register component
Screenshot 2025-07-24 223855

@Bala327
Copy link
Author

Bala327 commented Jul 24, 2025

kindly accept my pull request or any changes in the UI component kindly inform me. I will rectify as soon as possible

@Bala327
Copy link
Author

Bala327 commented Jul 25, 2025

@Ayush0316 kindly accept my pull request or any changes in the UI component kindly inform me. I will rectify as soon as possible

const AllRoutes = () => {
return (
<Routes>
<Route path="/register" element={<Register />} />
Copy link
Owner

Choose a reason for hiding this comment

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

Define this route in guest route similar to login route.

Copy link
Owner

Choose a reason for hiding this comment

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

Don't change anything in this file else it will give you merge conflicts when real landing page issue will be merged.

<p className="text-sm text-white/70 text-center mt-4">
Already have an account?{" "}
<span
onClick={() => navigate("/login")}
Copy link
Owner

Choose a reason for hiding this comment

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

As it is already mentioned in contributing.md, don't use magic strings, to navigate to login page use GUEST_PATH.LOGIN.

<div />

{/* Register Link */}
<p className="mt-4 text-sm text-blue-500 cursor-pointer" onClick={() => navigate("/register")}>
Copy link
Owner

Choose a reason for hiding this comment

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

Don't use magic string.

@KrishChothani
Copy link
Collaborator

Hey @Ayush0316
Resolve conflict

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.

3 participants