-
Notifications
You must be signed in to change notification settings - Fork 78
Test/auth signup validation cleanup #46
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
Test/auth signup validation cleanup #46
Conversation
✅ Deploy Preview for paisable ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Hey @archa8, Thanks for your feedback on the previous PR! I’ve cleaned up the branch, so this new PR only includes the essential code and test changes. Also, could you please add the 'hacktoberfest' label to this issue? That would be helpful. Appreciate your review! |
archa8
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kindly delete the package.json, package-lock.json (from the root) and .env from the frontend folder. Rest is fine
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep your editor code spacing from 4 to 2-space indentation.
It's very hard to review your changes, and it aligns with the project's formatting standards.
… removed unwanted files
|
@archa8 I’ve removed the requested files from the root and frontend folders and also applied the changes requested by @Avdhesh-Varshney. I’d appreciate it if you could review the updates. Your feedback would be very helpful. |
|
Hey @archa8, I’ve made the requested changes. Just wanted to check if you’ve had a chance to review them yet. |
archa8
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
|
@archa8 thank you, @Avdhesh-Varshney your review? |
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for contribution 🎉
Keep hacking & keep learning!
|
@Avdhesh-Varshney Thanks for merging! Appreciate the review |

Cleanup: test/auth-signup-validation branch formatting fixes
Description
This PR cleans up the
test/auth-signup-validationbranch by removing unnecessary formatting noise, including tab-to-space corrections and consistent use of single quotes in imports. No functional changes were made to the code or tests.Related Issue
Addresses previous PR #16 and cleanup comment; original issue #7
Motivation and Context
The previous PR included formatting changes (tabs, quotes) that made reviewing the actual test logic difficult. This cleanup makes the diff focused only on meaningful changes, improving readability and maintainability.
Types of Changes
How Has This Been Tested?
Screenshots (if applicable):
Not applicable (backend test files only).
Checklist