[Fix] 마이페이지 스크롤 개선 및 연결된 provider 모두 조회 - #361
Merged
Merged
Conversation
• 사용자가 인증된 경우 제공자 정보를 반환하는 API를 구현 • `GET` 요청을 통해 세션 정보를 확인하고, 사용자 ID에 기반하여 제공자 목록을 조회하는 로직 추가
• 로그인 정보 컴포넌트를 리팩토링하여 사용자 정보를 props로 받지 않도록 변경 • 제공자 정보를 API를 통해 동적으로 조회하고, 로그인 상태에 따라 표시하도록 구현
KimGeonWoo-p
approved these changes
Feb 3, 2026
Comment on lines
+39
to
+40
| {providers.map((provider, i) => { | ||
| const { name, icon, color } = getProviderInfo(provider); |
| export const getProviderInfo = (email?: string | null, image?: string | null) => { | ||
| if (!email) return { name: '이메일', icon: 'E', color: '#000' }; | ||
| export const getProviderInfo = (provider: string) => { | ||
| if (!provider) return { name: '이메일', icon: 'E', color: '#000' }; |
seongwon030
approved these changes
Feb 4, 2026
| const providers = await getProviders(); | ||
| setProviders(providers); | ||
| } catch (err) { | ||
| console.error(err); |
defender737
approved these changes
Feb 4, 2026
defender737
left a comment
defender737
left a comment
Collaborator
There was a problem hiding this comment.
로그인 정보에서 "로그인 중"이라는 멘트를 빼도 좋을 것 같습니다
• 제공자 정보를 가져오는 API 호출 시 에러 처리를 제거하여 코드 간결성 향상 • 비동기 함수 내에서 직접적으로 제공자 정보를 설정하도록 수정
• 로그인 중인 제공자의 이름만 표시하도록 변경하여 UI 간결성 향상 • 불필요한 텍스트를 제거하여 사용자 경험 개선
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
관련 이슈
close #356
완료 작업
overflow:hidden으로 되어있었던 로직을overflow:auto로 변경해 스크롤 문제 해결