ShwetKhatri2001 / AR-Webstore

Webstore built to showcase photorealistic 3D digital objects in AR
https://ar-webstore.netlify.app/
MIT License
58 stars 101 forks source link

fix: Added sign-in and sign-up #239

Closed IrfanshaikH-7 closed 8 months ago

IrfanshaikH-7 commented 8 months ago

Fixes #100

Changes: Added Sign-in and Sign-up pages

Screenshots: Screenshot 2024-01-23 185151 Screenshot 2024-01-23 185208

Please let me know if any changes are required

netlify[bot] commented 8 months ago

Deploy Preview for ar-webstore ready!

Name Link
Latest commit 95736309d5845aae2f77d02bfc0f9e7ee0d772e3
Latest deploy log https://app.netlify.com/sites/ar-webstore/deploys/65b28f3490df17000933c767
Deploy Preview https://deploy-preview-239--ar-webstore.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.

ShwetKhatri2001 commented 8 months ago

Add a link on the Navbar as Sign In so that I can check it in the deploy preview. In the image of the /sign-in page, I see the title Sign up, change it to Sign In.

Also, the yellow background color of the buttons isn't matching with our existing theme colors. Change it to something else like black or gray.

IrfanshaikH-7 commented 8 months ago

Changes added -

ShwetKhatri2001 commented 8 months ago

Routes are added in two files. Keep them in App.js. Currently, other pages are not accessible because of missing routes in index.js, make them accessible. Also, Keep the Navbar on top of sign-in and sign-up pages as well. It can be merged after that.

IrfanshaikH-7 commented 8 months ago

Routes are added in two files. Keep them in App.js. Currently, other pages are not accessible because of missing routes in index.js, make them accessible. Also, Keep the Navbar on top of sign-in and sign-up pages as well. It can be merged after that.

if you say so