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

feat(fe): settings page save #1994

Merged
merged 13 commits into from
Aug 26, 2024
Merged

feat(fe): settings page save #1994

merged 13 commits into from
Aug 26, 2024

Conversation

jihorobert
Copy link
Contributor

@jihorobert jihorobert commented Aug 22, 2024

Description

스크린샷 2024-08-26 오전 3 31 48

일반 Settings 페이지 작업 진행했습니다.

password만 바꾸는 것도 가능하고, name 혹은 major만 바꾸는 것도 가능합니다. 이때 name과 major만 바꾸고 싶으면 기존 password는 입력하지 않아도 됩니다. 기본값(Placeholder)으로는 기존의 회원정보를 토대로 입력되어있습니다.
작성중 이탈시 뜨는 모달창에 대해서는 디자인대로 되어있지 않고 일단 기본 팝업창을 이용하였습니다. 또한 사이트 내부에서 이동시에만 모달창이 뜨고있으며(ex. Settings페이지에서 Problem페이지로 이동시 등), 외부이동(창닫기, 새로고침, 뒤로가기)시에 모달이 뜨는 것에 대해서는 논의가 필요할 것 같아서 우선 막아놓았습니다.

스크린샷 2024-08-26 오전 3 36 17

closes TAS-521
closes TAS-520

Additional context

  • 해당 PR이 우선적으로 머지되고나서, '기존 회원정보수정 필수' 작업 마무리하겠습니다.
  • 리뷰 해주시고나서 승인하시면은 그때 주석들 지우고 제가머지할게요~!
  • majors 리스트는 constants파일에 따로 빼놨습니다.

Before submitting the PR, please make sure you do the following

Copy link

vercel bot commented Aug 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
codedang ✅ Ready (Inspect) Visit Preview Aug 26, 2024 5:28am

Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 34.9%, saving 1.08 KB.

Filename Before After Improvement Visual comparison
apps/frontend/public/invisible.png 1.66 KB 1.19 KB -28.0% View diff
apps/frontend/public/settings.png 1.43 KB 834 bytes -42.9% View diff

16 images did not require optimisation.

@jwoojin9
Copy link
Contributor

형, 새벽까지 불태워서 만들었어... 궁금한 거 있으면 물어봐! (근데 늦게 일어나서 일찍 확인 못할 수 있어...)

Copy link

Copy link

Copy link
Member

@jimin9038 jimin9038 left a comment

Choose a reason for hiding this comment

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

👍

@jimin9038 jimin9038 merged commit 56c2216 into main Aug 26, 2024
10 checks passed
@jimin9038 jimin9038 deleted the t521-settings-page branch August 26, 2024 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done ✔️
Development

Successfully merging this pull request may close these issues.

3 participants