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

Addition of profile and logout option #615

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

Conversation

PriyanshiAgr
Copy link

@PriyanshiAgr PriyanshiAgr commented Oct 16, 2024

Improved Body Content UI Across All Pages #348

Description

  1. UI Changes on the Home Page, Collection Page, About Us Page, Services Page, Contact Us Page, Login & SignUp Page, and Cart Page (theme, font, looks, new features)
  2. Also created a logo for it
  3. Added Profile feature after user login or signup
  4. Added Logout feature
  5. Inserted better-quality images

Related Issues

Type of PR

  • (New Feature)
  • (UI enhancement)

Screenshots/videos (if applicable)

screencapture-127-0-0-1-3000-1-IndexPage-index-html-2024-10-16-16_20_42

screencapture-127-0-0-1-3000-2-CollectionsPage-menu-html-2024-10-16-16_33_48

screencapture-127-0-0-1-3000-9-Cart-cart-html-2024-10-16-16_34_48

screencapture-127-0-0-1-3000-7-Login-SignUpPage-Login-SignUp-html-2024-10-16-16_21_38

screencapture-127-0-0-1-3000-8-Profile-profile-html-2024-10-16-16_36_02

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I have performed a self-review of my code
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

I made changes on all the pages of the site, as I mentioned in my issue #348

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @PriyanshiAgr! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Copy link

netlify bot commented Oct 16, 2024

Deploy Preview for rococo-mooncake-0d89f7 ready!

Name Link
🔨 Latest commit 26f2e7d
🔍 Latest deploy log https://app.netlify.com/sites/rococo-mooncake-0d89f7/deploys/671d32fe079a70000844ab04
😎 Deploy Preview https://deploy-preview-615--rococo-mooncake-0d89f7.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.

@Anjaliavv51
Copy link
Owner

Anjaliavv51 commented Oct 17, 2024

@PriyanshiAgr resolve merge conflicts.

@Anjaliavv51
Copy link
Owner

Anjaliavv51 commented Oct 17, 2024

@PriyanshiAgr instead this all , just add logout and profile option to our webpage , coz these are really great , but i didn't like theme soo

create another folder like , another theme for retro site orelse instead , theme of entire webite changes , into current and ur theme according to user's wish

@Anjaliavv51 Anjaliavv51 changed the title Improved Body Content UI Across All Pages #348 Addition of profile and logout option Oct 17, 2024
@PriyanshiAgr
Copy link
Author

PriyanshiAgr commented Oct 17, 2024

OK..
so concluding,

  1. Add a profile and logout feature to the original site.
  2. Add my theme as a switch button, so the user can change the site theme (if he wants to) (if possible)

I hope this is what you saying..

@ambrishgoswami
Copy link

I would like to work on this issue. Kindly assign me the above task

@PriyanshiAgr
Copy link
Author

@Anjaliavv51 added the profile and logout feature.. check it out !!

@PriyanshiAgr
Copy link
Author

Hey @Anjaliavv51 !.
My scores on the leaderboard have not been updated.
May I know why ?? ;(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Addition of profile and logout option
3 participants