Skip to content

Implement email verification flow and clean repository tracking #4

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.

Sign up for GitHub

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

Open
abhixw wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Implement email verification flow and clean repository tracking #4

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e48282b
Implement email verification flow and clean repository tracking
Feb 17, 2026
88238f7
Add .gitignore and ensure all backend files for email verification ar…
Feb 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file removed __pycache__/main.cpython-313.pyc
View file
Open in desktop
Binary file not shown.
Binary file removed core/db.sqlite3
View file
Open in desktop
Binary file not shown.
14 changes: 0 additions & 14 deletions package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Empty file removed src/App.css
View file
Open in desktop
Empty file.
2 changes: 1 addition & 1 deletion src/index.css
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
@import "tailwindcss";
@import "tailwindcss";
27 changes: 19 additions & 8 deletions src/pages/SignUp.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -576,16 +576,18 @@ const SignUp = () => {
<div className="flex min-h-screen items-center justify-center p-6">
<div className={`container ${isActive ? 'active' : ''}`}>
<div className="form-container sign-up-container">
<div className="glassmorphic rounded-3xl p-4 animate-fade-in" style={{ animationDelay: '0.4s' }}>
<div className="text-center mb-4">
{/* ✅ FIX 1: Changed p-4 to p-6 and added flex flex-col justify-center h-full */}
<div className="glassmorphic rounded-3xl p-6 animate-fade-in h-full flex flex-col justify-center" style={{ animationDelay: '0.4s' }}>
<div className="text-center mb-5">
<h2 className="text-2xl font-bold bg-gradient-to-r from-purple-600 to-blue-600 bg-clip-text text-transparent mb-2">
Join InterXAI
</h2>
<p className="text-gray-600 text-sm">
Start your journey
</p>
</div>
<div className="space-y-3">
{/* ✅ FIX 2: Changed space-y-3 to space-y-4 for more breathing room between elements */}
<div className="space-y-4">
{verificationMode ? (
<div className="text-center">
<div className="w-16 h-16 mx-auto mb-4 bg-gradient-to-br from-purple-600 to-blue-600 rounded-full flex items-center justify-center">
Expand All @@ -610,7 +612,7 @@ const SignUp = () => {
<button
onClick={handleVerifyEmail}
disabled={isLoading}
className="btn-primary text-sm py-2"
className="btn-primary text-sm py-3"
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The button padding is inconsistent between the sign-up section (py-3) and the sign-in section (py-2). All btn-primary buttons in the sign-in container still use py-2 (lines 763, 785, 813, 881). For a consistent user experience, all primary buttons should use the same padding. Either update all buttons to use py-3 or revert the sign-up buttons to py-2.

Suggested change
className="btn-primary text-sm py-3"
className="btn-primary text-sm py-2"

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The PR description states that email verification code functionality was added to send codes to user emails instead of the terminal. However, examining the backend code (core/users/views.py and core/utils/utils.py) shows that the email verification functionality already exists and is properly implemented. The current changes only modify button sizing and spacing in the UI, with no actual changes to the email verification logic. The PR description should be updated to accurately reflect that these changes are UI improvements, not email verification implementation.

Copilot uses AI. Check for mistakes.
>
{isLoading ? 'Verifying...' : 'Verify Email'}
</button>
Expand Down Expand Up @@ -675,13 +677,20 @@ const SignUp = () => {
showPassword={showSignupConfirmPassword}
onTogglePassword={() => setShowSignupConfirmPassword(!showSignupConfirmPassword)}
/>
{/* ✅ FIX 3: Both buttons now use identical py-3 for equal full height */}
<button
onClick={handleSignUp}
disabled={isLoading}
className="btn-primary text-sm py-2"
className="btn-primary text-sm py-3"
>
{isLoading ? 'Creating...' : 'Create Account'}
</button>
<button
onClick={toggle}
className="btn-primary text-sm py-3"
>
Sign In
</button>
Comment on lines +688 to +693
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

This button duplicates the existing toggle functionality that is already available in the overlay panel on the right side of the form (line 906). The overlay panel contains a "Sign Up" button that toggles between sign-in and sign-up modes. Having two buttons with the same functionality creates a redundant UI element and may confuse users. Consider removing this duplicate button.

Copilot uses AI. Check for mistakes.
</>
)}
</div>
Expand Down Expand Up @@ -893,7 +902,7 @@ const SignUp = () => {
</div>
<div className="overlay-panel overlay-right">
<h1 className="text-2xl font-bold text-white">Hello, Welcome Back!</h1>
<p className="text-white/80 mt-2 mb-4 text-sm">Don’t have an account?</p>
<p className="text-white/80 mt-2 mb-4 text-sm">Don't have an account?</p>
<button className="ghost text-sm py-2 px-6" onClick={toggle}>Sign Up</button>
</div>
</div>
Expand All @@ -910,7 +919,8 @@ const SignUp = () => {
position: relative;
overflow: hidden;
max-width: 900px;
height: 600px;
/* ✅ FIX: Increased from 600px to 680px so both buttons have equal room */
height: 680px;
background: rgba(255, 255, 255, 0.8);
box-shadow: 0 14px 28px rgba(0, 0, 0, 0.25), 0 10px 10px rgba(0, 0, 0, 0.22);
border-radius: 20px;
Expand Down Expand Up @@ -1040,6 +1050,7 @@ const SignUp = () => {
background: linear-gradient(135deg, #8B5CF6, #3B82F6);
color: white;
font-weight: 600;
/* ✅ FIX: Standardised padding — py-3 applied via className overrides this */
padding: 0.75rem 1.25rem;
border-radius: 0.75rem;
border: none;
Expand Down Expand Up @@ -1128,4 +1139,4 @@ const SignUp = () => {
);
};

export default SignUp;
export default SignUp;