-
Notifications
You must be signed in to change notification settings - Fork 29
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
add signup page #90
add signup page #90
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hi @Vin-Jex, I’ve created this pull request with the following updates:
If anything needs to be changed or improved on the signup page, please let me know. I’m happy to make adjustments based on your feedback! Thanks! |
Here’s a rephrased and more structured version that communicates the feedback clearly and professionally: Hi @Ryadav0654 Thank you for your contribution! I’ve reviewed your pull request, and I appreciate the work you’ve done so far. Here’s some feedback on the signup page:
Let me know if you have any questions or need further clarification. I’m happy to assist! |
Please resolve this issue quick else I would close this PR and make the issue opened for grab again. |
Ok, I am working. |
Hi @Vin-Jex , I've addressed all the issues and applied the necessary styling changes. Please review the latest updates and let me know if you have any further suggestions or improvements. |
Firsly, you failed to fix the bug I raised previously and lastly, this branch now has a merge conflict. So resolve the bug, update you branch and fix merge conflict. Thank you! |
@Ryadav0654 is attempting to deploy a commit to the VinJex Team on Vercel. A member of the Team first needs to authorize it. |
fixes: #85
Screenshot
signUp.page.mp4