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

[FE] 마이페이지 화면 퍼블리싱 #839

Merged
merged 2 commits into from
Dec 5, 2024
Merged

[FE] 마이페이지 화면 퍼블리싱 #839

merged 2 commits into from
Dec 5, 2024

Conversation

jinhokim98
Copy link
Contributor

issue

구현 사항

마이페이지 퍼블리싱 작업 수행

image

로그인 후 사용자가 혜택을 누릴 수 있는 화면, 마이페이지 화면 퍼블리싱 작업만 수행했습니다.
이 페이지에서 로그아웃, 닉네임 설정, 계좌번호 설정, 내가 만든 행사 목록, 탈퇴 기능을 수행할 수 있습니다.
피그마에 있는 레이아웃 대로 작업했으며 기능은 추후에 추가할 예정입니다.

🫡 참고사항

@jinhokim98 jinhokim98 added this to the v2.2.0 milestone Nov 25, 2024
@jinhokim98 jinhokim98 self-assigned this Nov 25, 2024
@jinhokim98 jinhokim98 linked an issue Nov 25, 2024 that may be closed by this pull request
1 task
Copy link

Copy link
Contributor

@soi-ha soi-ha left a comment

Choose a reason for hiding this comment

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

멋집니댱, 쿠키! 🍪

Comment on lines +30 to +41
<TextButton textColor="onTertiary" textSize="body">
닉네임 설정하기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
기본 계좌 번호 설정하기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
내가 만든 행사 목록 보기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
탈퇴하기
</TextButton>
Copy link
Contributor

Choose a reason for hiding this comment

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

나중에 기능 추가할 때, 해당 항목들과 이동 Url을 배열로 정리해서 렌더링시키는 것도 좋을 것 같아요~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋아요~~ 아직은 퍼블리싱 단계라 기능을 넣을 때 도입해볼게요!

Copy link
Contributor

Choose a reason for hiding this comment

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

탈퇴하기 부분은 충분히 destructive한 기능이니까 color를 destructive로 적용해 줘도 좋을 것 같아요

Copy link
Contributor

@pakxe pakxe left a comment

Choose a reason for hiding this comment

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

좋아요좋아요~ 고생많았습니다. 쿠키~ 저도 남은 기능을 이번 주에 많이 개발해볼게요!

Comment on lines +7 to +13
<Flex
flexDirection="column"
width="100%"
gap="0.5rem"
backgroundColor="white"
padding="1rem"
css={{borderRadius: '0.75rem'}}
Copy link
Contributor

@pakxe pakxe Dec 3, 2024

Choose a reason for hiding this comment

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

Flex 컴포넌트가 사용되는 모습을 보니 어색하네요. 간편하게 쓰기 위해서 여러 props를 열어두었는데, css prop도 받는걸 보니 누군가는 '어라 그냥 css에 다 집어넣으면 되지 왜 따로갈비지?'라고 생각할 수도 있을 것 같아서요.. 다양한 사용 방법을 모두 수용하다보니 이렇게 된 것 같기도 하구요. 어떻게 만드는게 좋았을 지 고민이 되네요. 고쳐달라는 뜻은 아니고 아마 저와 같은 생각을 하는 사람이 있을 것 같아 적어봤써요 😭

Copy link
Contributor

Choose a reason for hiding this comment

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

저도 어느정도 동감해요! css를 인자로 받아서, 다른 component처럼 css를 넘겨줄 수 있도록 하는게 제일 깔끔해 보이기는 해요!
좋은 방법이 있을지 고민해 보도록 하겠습니다 :)

Copy link
Contributor

@Todari Todari left a comment

Choose a reason for hiding this comment

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

고생많았습니다~~! :)

Comment on lines +7 to +13
<Flex
flexDirection="column"
width="100%"
gap="0.5rem"
backgroundColor="white"
padding="1rem"
css={{borderRadius: '0.75rem'}}
Copy link
Contributor

Choose a reason for hiding this comment

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

저도 어느정도 동감해요! css를 인자로 받아서, 다른 component처럼 css를 넘겨줄 수 있도록 하는게 제일 깔끔해 보이기는 해요!
좋은 방법이 있을지 고민해 보도록 하겠습니다 :)

Comment on lines +30 to +41
<TextButton textColor="onTertiary" textSize="body">
닉네임 설정하기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
기본 계좌 번호 설정하기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
내가 만든 행사 목록 보기
</TextButton>
<TextButton textColor="onTertiary" textSize="body">
탈퇴하기
</TextButton>
Copy link
Contributor

Choose a reason for hiding this comment

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

탈퇴하기 부분은 충분히 destructive한 기능이니까 color를 destructive로 적용해 줘도 좋을 것 같아요

Copy link

github-actions bot commented Dec 5, 2024

@jinhokim98 jinhokim98 merged commit f50f4d5 into fe-dev Dec 5, 2024
2 checks passed
@jinhokim98 jinhokim98 deleted the feature/#838 branch December 5, 2024 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[FE] 마이페이지 화면 퍼블리싱
4 participants