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.

Sign up for GitHub

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

Jump to bottom

[Refactor] 멘토님 코드 리뷰용 PR 생성 #145

Closed
wants to merge 514 commits into from
Closed

[Refactor] 멘토님 코드 리뷰용 PR 생성 #145

wants to merge 514 commits into from

Conversation

nakyeonko3
Copy link
Collaborator

@nakyeonko3 nakyeonko3 commented Sep 9, 2024 •
edited by github-actions bot
Loading

🚀 코드 리뷰용 PR

Summary by CodeRabbit

## Release Notes

### New Features
- **PlaylistDetailPage**: 추가된 새로운 페이지로, 특정 플레이리스트의 세부 정보를 보여주고 수정할 수 있는 기능 제공.
- **PostDetailPage**: 게시물과 댓글 섹션을 표시하는 새로운 페이지 추가.
- **PrivacyPolicyPage**: 개인정보처리방침 페이지 추가.
- **ProfilePage & ProfileEditPage**: 사용자 프로필 보기 및 편집 기능 추가.
- **SearchPage**: 검색 결과를 포스트 및 사용자로 필터링하는 기능 추가.
- **SelectPliPage & SelectVideoPage**: 선택된 동영상을 처리하는 로직 추가.
- **SettingsPage**: 설정 페이지 리팩터링 및 로그아웃 기능 추가.
- **SignInPage**: Google 로그인 기능 포함한 새로운 로그인 페이지 추가.
- **TermsOfServicePage**: 서비스 이용약관 페이지 추가.

### Refactor
- **ProtectedRoute & PublicRoute**: Firebase 인증 상태 확인 및 라우팅 로직 개선.

devdeun and others added 30 commits September 5, 2024 23:31
@devdeun
design: Video border 스타일 수정 (#82)
24153c0
@nakyeonko3
feat: 코맨트 CRUD fetch 추가
1d0be6f
@devdeun
feat: Spinner 컴포넌트 추가 (#85)
08eff1b
@devdeun
feat: Loading 컴포넌트 추가 (#85)
52ab3db
@devdeun
feat: loading, Spinner 컴포넌트 적용 (#85)
00bc9a9
@devdeun
bug: 포스트 작성시간 오류 수정 (#87)
d2ae113
@devdeun
chore: console log 삭제 (#87)
0e6c8b8
@HSjjs98
Merge pull request #86 from Dev-FE-1/enhancement/loading-page-85
5019816 
[Enhancement] Spinner 및 Loading 컴포넌트 추가
@HSjjs98
Merge pull request #83 from Dev-FE-1/design/profile-page-82
49b6bec 
[Design] 프로필 페이지 및 공통 스타일 수정
@HSjjs98
Merge pull request #88 from Dev-FE-1/bug/post-createdAt-87
400a67b 
[Bug] 포스트 작성일 오류 수정
@devdeun
chore: 오타 수정 (#89)
e71423b
@devdeun
feat: follower, following 클릭에 따라 탭 활성화 (#89)
491939d
@devdeun
design: Loading 컴포넌트 스타일 수정 (#89)
381ad6a
@HSjjs98
feat: AddVideos 페이지 추가에 따른 경로 설정 (#74)
48da58a
@HSjjs98
feat: 주석 제거 및 Usermodel에 subscriptions 추가 (#74)
027cb57
@HSjjs98
feat: 플레이리스트 수정, 삭제, 구독 기능 추가 (#74)
dc85096
@HSjjs98
Merge branch 'develop' of https://github.com/Dev-FE-1/Toy_Project_3_t…
d5afd28 
…eam4 into enhancement/add-videos-to-playlist-74
@nakyeonko3
fix: 코맨트 수정 안되는 버그 해결(#84)
3098e59
@devdeun
fix: 로그인한 유저 기준으로 팔로우/팔로잉 버튼이 뜨도록 수정 (#89)
8aec474
@devdeun
feat: 팔로잉 팔로워 탭 변경시 최신 데이터 불러오기 (#89)
b338b10
@nakyeonko3
feat: 대댓글 기능 추가, CommentModel 파일 위치 변경(#84)
23f7737
@HSjjs98
chore: lefthook 추가 (#74)
f1da5a3
@devdeun
Merge pull request #90 from Dev-FE-1/enhancement/following-follower-89
3e92c2e 
[Enhancement] 팔로우 페이지 기능 및 스타일 수정
@HSjjs98
style: 주석 제거, 오타 수정, 스타일 변경 (#74)
804f151
@HSjjs98
feat: 커스텀 훅 추가, 삭제, 위치 변경 (#74)
72c34aa
@HSjjs98
remove: 더미 삭제 (#74)
abf2a75
@nakyeonko3
style: 주석, 띄어쓰기 제거
82e230e
@HSjjs98
feat: 타입 변경에 따른 코드 수정 (#74)
cfa9f0e
@nakyeonko3
feat: formatRelativeDate에 '방금 전' '분전' 추가
78f1b4d
@HSjjs98
feat: 플레이리스트 수정, 삭제, 비디오 추가, 비디오 삭제 기능 구현 (#74)
b3569bb
@nakyeonko3 nakyeonko3 mentioned this pull request Sep 12, 2024
Closed
1 task
nakyeonko3 and others added 28 commits September 13, 2024 02:02
@nakyeonko3
refactor: Post.tsx 리펙토링(#163)
01abc50 
- videoId를 useMemo를 사용하여 video.post가 바뀔 때마 비디오 아이디를
  fetch하도록 수정함.
- tanstackQuery의 mutation을 사용하는 객체는 뒤에 mutation을 붙여서
  구분함.
- bookMark, isClickable, label, isPublicPlaylist를 프로퍼티를 가진
  playlistObj를 생성함.
@nakyeonko3
refactor: useFetchVideoTitle가 useQuery를 사용하도록 수정(#160)
3b0e3a9 
- 동영상 타이틀이 랜더링되는 속도가 향상되어 비디오 제목이 unknown으로 최대한 나오지 않게함.
- useQuery를 사용하여 동일한 fetch를 여러번 하지 않고 캐싱을 함.
- fetchYouTubeVideoData에서 try-catch문을 사용하여 reponse.data()시의 에러도 로깅하도록함.
- 에러 로그에 status HTTP 상태 코드를 넣어 더 나오는 에러로깅을 하도록함.
@nakyeonko3
refactor: logo.svg를 preload가 되도록 수정(#160)
9d2fffc 
- 가끔 이미지가 뜨지 않는 경우가 있음.
- logo.svg를 우선적으로 로드 시켜 "플리 로고" 가 보이지 않고 로고 이미지가 빠르게 랜더링 되도록함.
@devdeun
feat: 댓글 글자수 제한 적용 (#161)
47a5fa2
@devdeun
Merge pull request #164 from Dev-FE-1/enhancement/input-max-length-161
0a12d11 
[Enhancement] Input 값 최대 길이 설정
@devdeun
Merge pull request #162 from Dev-FE-1/bug/likes-error-160
203d2bb 
[Bug] 좋아요 기능 관련 문제 해결, Post.tsx 리팩토링, CommentSection 리팩토링
@HSjjs98
feat: 모달을 훅으로 관리 (#158)
58d9001
@HSjjs98
chore: npm audit fix (#158)
070c6e2
@HSjjs98
feat: NewPost를 fullmodal을 사용하여 보여주기 & 프로필 수정 페이지 경로 수정 (#158)
511bacf
@HSjjs98
Merge branch 'develop' of https://github.com/Dev-FE-1/Toy_Project_3_t…
a62635e 
…eam4 into enhancement/post-add-to-modal-158
@HSjjs98
feat: 오버레이 관련 코드 컴포넌트로 분리 (#158)
633529f
@HSjjs98
feat: fullmodal 추가, 모달 전역 상태 관리, useModalWithOverlay 훅 추가 (#158)
da0f890
@HSjjs98
feat: 모달 내부에서 헤더가 랜더링 될 . 수있도록 usePortal 관련 코드 추가 (#158)
185db69
@HSjjs98
refactor: 훅으로 모달 상태 관리하도록 코드 변경 (#158)
dc12b39
@HSjjs98
refactor: 훅 관련 코드 추가 및 삭제 (#158)
7e88097
@HSjjs98
refactor: 경로 설정 관련 코드 변경 또는 추가 (#158)
88d7500
@devdeun
Merge pull request #170 from Dev-FE-1/enhancement/post-add-to-modal-158
803ab1d 
[Enhancement] 포스트 추가 페이지들을 모달로 처리 및 모달과 오버레이를 전역으로 상태 관리하도록 변경
@HSjjs98
refactor: 모달 동작 수행 및 기능 관련 코드 수정 (#171)
dcf6142 
- 포스트 추가하기 첫번째 모달 close 헤더로 바꾸기, 완료 버튼을 다음으로 수정
- 남의 플리에 있는 비디오 내플리에 추가하기 selectpli 뒤로가기 경로 수정
- 포스트 추가하기 기능 모달 동작 관련 수정
- 링크로 포스트 추가하기에서 더이상 url을 사용하지 않고 비디오와 플레이리스트 정보를 전달
@HSjjs98
Merge pull request #172 from Dev-FE-1/refactor/bugs-fix-171
b68dd5a 
[Refactor] 모달 관련 동작 수정 및 헤더 제목 등 수정
@HSjjs98
fix: 프로필 페이지에서 포스트와 좋아요 탭에서 좋아요가 바로 적용안되는 문제 해결 (#174)
98b4a4b 
- 추가로 좋아요 탭에서 좋아요 토글될때 포스트가 바로 사라지는 문제 수정
@HSjjs98
feat: 이미 플레이리스트에 포함된 비디오 추가 불가하도록 수정 (#174)
70cb736
@devdeun
Merge pull request #175 from Dev-FE-1/bug/profile-likes-174
36bee5e 
[Bug]프로필 페이지에서 포스트와 좋아요 탭에서 좋아요가 바로 적용안되는 문제 해결
@HSjjs98
Merge branch 'main' of https://github.com/Dev-FE-1/Toy_Project_3_team4 …
04e3975 
…into develop
@devdeun
Merge pull request #176 from Dev-FE-1/merge/merge-branch
d72c5c9 
[Merge] 머지용 브랜치 생성
@HSjjs98
Merge branch 'main' into develop
55bdfee
@HSjjs98
Merge pull request #173 from Dev-FE-1/develop
b4692b8 
메인으로 머지
@HSjjs98
feat: 다름 사람의 플리에서 비디오를 내 플리에 추가할때 중복된 비디오가 추가되지 않도록 수정 (#177)
b7aaf7b
@HSjjs98
Merge pull request #178 from Dev-FE-1/enhancement-videos-add-177
053d654 
[Enhancement] 다름 사람의 플리에서 비디오를 내 플리에 추가할때 중복된 비디오가 추가되지 않도록 수정
@nakyeonko3 nakyeonko3 closed this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dozukwang dozukwang dozukwang left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nakyeonko3 @dozukwang @devdeun @HSjjs98 @wonjichoe