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(@depromeet/toks-components/ToksHeader): 헤더에 로그인 안되어있을시 프로필 버튼 안나오게 변경 #257

Merged
merged 4 commits into from
Mar 1, 2023

Conversation

dengoyoon
Copy link
Collaborator

💡 왜 PR을 올렸나요?

  • 프로필 버튼이 로그인이 안되어있다면 아예 안나오게 변경하였습니다.
  • 즉, 프로필 버튼에 "로그인"이라고 나올일이 없습니다. (해당 부분을 null로 반환되게 처리)
  • 혹시나 유저가 의도적으로 로그인 된 상태에서 login url로 접근하면 프로필 버튼이 표시되어서 로그아웃을 할 수 있습니다.
  • 로그인 화면 로딩시 Suspense fallback에서 반환하는 프로필 버튼이 기본 버튼으로 잠깐 나오는 현상이 있어 아예 안보여지게 변경하였습니다
  • 디자인 수정에 따라 로그인 화면의 로그인 버튼을 general -> primary로 변경하였습니다

스크린샷 2023-02-27 오후 1 17 20

스크린샷 2023-02-27 오후 1 16 47

💁 무엇이 어떻게 바뀌나요?

  • 디자인 레퍼런스:
  • 관련 슬랙 링크:

💬 리뷰어분들께

생각보다 엄청 작업이 단순했네요 ,, 늦장피우다가 지금 올립니다 🙂..

Copy link
Member

@minsgy minsgy left a comment

Choose a reason for hiding this comment

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

👍

@dengoyoon dengoyoon merged commit 2f1ef74 into master Mar 1, 2023
@dengoyoon dengoyoon deleted the feat/header branch March 1, 2023 07:13
chaaerim pushed a commit that referenced this pull request Mar 1, 2023
…오게 변경 (#257)

* ✨ feat: 로그인이 안되어있으면 프로필 버튼 안보이게 변경

* 🎨 style: Suspense Skeleton에서 기본 프로필 이미지 나오는 현상 제거

* 🎨 style: 로그인 화면 배너 로그인 버튼 색상 primary로 변경
LineGu pushed a commit that referenced this pull request Mar 5, 2023
* ✨ feat: 종료된 스터디 입장 가능하게 변경

* feat(@depromeet/toks-components, quiz) : Avatar 컴포넌트 리팩토링 (#260)

* ♻️ refactor: Avatar 리팩토링 및 툴팁 적용 구현

* ♻️ refactor: Avatar 컴포넌트 다시 적용하기

* ♻️ refactor: view 기본 값 5로 정의

* feat(@depromeet/toks-components/ToksHeader): 헤더에 로그인 안되어있을시 프로필 버튼 안나오게 변경 (#257)

* ✨ feat: 로그인이 안되어있으면 프로필 버튼 안보이게 변경

* 🎨 style: Suspense Skeleton에서 기본 프로필 이미지 나오는 현상 제거

* 🎨 style: 로그인 화면 배너 로그인 버튼 색상 primary로 변경

* ♻️ refactor: 문자열 변경

---------

Co-authored-by: Minseok Choi <alstjrchl97@likelion.org>
Co-authored-by: DuHyeon Yoon <engus927@naver.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants