Skip to content
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.

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

Implement Login/Signup Page with Backend Authentication #184

Closed
wants to merge 0 commits into from

Conversation

saras-69
Copy link
Contributor

@saras-69 saras-69 commented Oct 3, 2024

#116 Description: This pull request introduces a login and signup page that integrates with the backend for user authentication. The implementation ensures secure user registration and login processes.

Key Features:

Developed a responsive login and signup interface.
Implemented form validation for user inputs (e.g., email format, password strength).
Integrated with backend authentication API for user registration and login.
Handled error messages and success notifications based on authentication outcomes.
Included functionality to persist user sessions after login.
Testing:

Conducted unit tests for form validation and authentication logic.
Manually tested the login and signup processes to verify correct functionality.
Ensured appropriate error handling for invalid inputs and authentication failures.

Copy link

netlify bot commented Oct 3, 2024

Deploy Preview for pathsphere ready!

Name Link
🔨 Latest commit d0b940c
🔍 Latest deploy log https://app.netlify.com/sites/pathsphere/deploys/670ad2611ffd410008e6ed30
😎 Deploy Preview https://deploy-preview-184--pathsphere.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aditya-bhaumik
Copy link
Owner

@saras-69 dont create pull request without assignment i have already assigned to someone else

This is not good open source practice

@saras-69
Copy link
Contributor Author

saras-69 commented Oct 3, 2024

Hey @aditya-bhaumik I apologize for the oversight. I understand the importance of proper task assignment. However, I would appreciate it if you could review my code for the Login/Signup page with backend authentication. I believe it meets the necessary requirements and can be a valuable addition to the project.

@aditya-bhaumik
Copy link
Owner

@saras-69 it is not good open source practice it will be unfair for the person who i had assigned this task

@aditya-bhaumik
Copy link
Owner

Still if he she is not able to complete the task in next 2 days will reopen this and merge it

@aditya-bhaumik aditya-bhaumik added the invalid This doesn't seem right label Oct 3, 2024
@saras-69
Copy link
Contributor Author

saras-69 commented Oct 3, 2024

Sure, @aditya-bhaumik! I wanted to check in and see if you have any feedback on my implementation. I’m eager to hear your thoughts!

@saras-69
Copy link
Contributor Author

saras-69 commented Oct 3, 2024

Btw Thank you for your quick response

@aditya-bhaumik
Copy link
Owner

Join discord i am helping there

@aditya-bhaumik
Copy link
Owner

Link is in contributing.md

@saras-69
Copy link
Contributor Author

saras-69 commented Oct 9, 2024

Hey @aditya-bhaumik, It’s been almost a week, and I noticed that this has not been implemented in the project yet. Could you please provide me with any updates?

@aditya-bhaumik
Copy link
Owner

resolve merge conflicts

@aditya-bhaumik
Copy link
Owner

image
also improve the ui please

@saras-69
Copy link
Contributor Author

Okay

@aditya-bhaumik
Copy link
Owner

please complete by tomorrow or i am closing this

@saras-69
Copy link
Contributor Author

please complete by tomorrow or i am closing this

Okay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants