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.

Sign up for GitHub

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

Jump to bottom

Add page account security and update page account profile #267

Merged
merged 3 commits into from
Oct 16, 2023
Merged

Add page account security and update page account profile #267

merged 3 commits into from
Oct 16, 2023

Conversation

billalxcode
Copy link
Contributor

Account Security

Screenshot (61)

What do I do?

  1. Create an account security page
  2. Add account security to the sidebar menu

Account Profile

image

What was fixed?

  1. Removed the Upload Image button
  2. Added birthday date input
  3. Added gender selection input
  4. Add name and bio below profile photo

@netlify
Copy link

netlify bot commented Oct 15, 2023 •
edited
Loading

✅ Deploy Preview for comforting-cocada-22086c ready!

Name Link
🔨 Latest commit 155070d
🔍 Latest deploy log https://app.netlify.com/sites/comforting-cocada-22086c/deploys/652d296a0c679b0008e198dd
😎 Deploy Preview https://deploy-preview-267--comforting-cocada-22086c.netlify.app/account-profile
📱 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.

@alfianchii alfianchii self-requested a review October 15, 2023 14:32
alfianchii
alfianchii requested changes Oct 15, 2023
View reviewed changes
Copy link
Collaborator

@alfianchii alfianchii left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image

Could you create a responsive UI for the Profile? And also if it pleases you, could you consider adding a feature to display/hide the password into Account Security page?

@billalxcode
Copy link
Contributor Author

billalxcode commented Oct 16, 2023 •
edited
Loading

Hello, I have fixed this page to be compatible with multiple devices like Desktop, Tablet and Mobile

Desktop Mode

Screenshot (64)

Tablet Mode

Screenshot (65)

Mobile Mode

Screenshot (66)

alfianchii
alfianchii approved these changes Oct 16, 2023
View reviewed changes
Copy link
Collaborator

@alfianchii alfianchii left a comment

Choose a reason for hiding this comment

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

TYSM!!

@alfianchii alfianchii merged commit ba30fbb into zuramai:main Oct 16, 2023
@billalxcode billalxcode deleted the billalxcode/account-security branch October 16, 2023 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alfianchii alfianchii alfianchii approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@billalxcode @alfianchii