Skip to content

Conversation

@gustn99
Copy link
Collaborator

@gustn99 gustn99 commented Jan 5, 2025

주요 작업 내용

  • 린트 및 컬러시스템 적용 관련 버그 일괄 해결했습니다.
  • getCurrentUserId로 사용하는 모든 id에 대한 변수명을 getCurrentUserId로 수정했습니다.

기타 작업 내용

  • 기존에 다른 유저 프로필 경로였던 /users 경로가 일부 남아있어 제거했습니다.
  • StyledText의 lineHeight 전부 삭제했습니다.
  • Feed 컴포넌트에서 dayjs의 fromNow를 사용할 때 relativeTime을 사용하라는 오류가 있어서 추가했는데, PostBase 컴포넌트에서는 relativeTime 없이도 fromNow를 사용하는 데 문제가 없어 조금 더 살펴봐야 할 것 같습니다.
  • Profile 컴포넌트에서 PostsContainer 전체에 cursor pointer 속성이 적용되어 '게시물이 없습니다'와 같은 텍스트가 렌더링될 때에도 커서가 포인터로 되는 문제를 발견했습니다. PostItem 컴포넌트에서 아이템당 cursor pointer 속성이 적용되기 때문에, 불필요한 속성으로 판단하여 제거했습니다.
  • PostItem, UserProfile 등 Profile에서만 사용하는 컴포넌트를 Profile 폴더 내부로 이동했습니다.
  • 그 외에 일부 눈에 띄는 코드 흐름 수정했습니다.

코드 리뷰 포인트

  • 일부 의존성이 수정된 것으로 보입니다. 풀 받으신 후 yarn install을 통해 의존성 통일 부탁드립니다.

작업 화면

  • 없음

@gustn99 gustn99 added fix Fix bugs that don't work feat New feature labels Jan 5, 2025
@gustn99 gustn99 self-assigned this Jan 5, 2025
@gustn99 gustn99 merged commit ba87c20 into dev Jan 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature fix Fix bugs that don't work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants