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

[enhancement] 댓글 페이지 생성 #92

Merged
merged 21 commits into from
Sep 6, 2024
Merged

Conversation

nakyeonko3
Copy link
Collaborator

@nakyeonko3 nakyeonko3 commented Sep 6, 2024

  • 기존에 Comment 타입 삭제 CommentModel 추가함.

🚀 풀 리퀘스트 제안

@coderabbitai: i🤷‍♀️gnore

📋 작업 내용

  • feat: 댓글 페이지 생성, App.tsx 수정, path.tsx에 댓글 페이지 경로 추가.
  • feat: 댓글 CRUD fetch 함수 추가
  • refactor: comment 타입 변경. 기존 코맨트 타입 삭제후 CommentModel 추가
    image
    image
  • fix: 코맨트 개수를 가져오지 못하는 버그 해결. getCommentsCount 함수를 이용해 코맨트 개수를 가져오도록함.
  • fix: 우분투 환경에 필요한 npm 종속성들을 추가함
export const getCommentsCount = async (postId: string): Promise<number> => {
  const postDoc = doc(postsCollection, postId);
  const commentsCollection = collection(postDoc, 'comments');
  const q = query(commentsCollection);

  const querySnapshot = await getDocs(q);
  return querySnapshot.size;
};

🔧 변경 사항

  • 📦 package.json, @rtsao/scc 종속성 추가함.
  • 📦 package.lock.json 변경
  • 🧹 eslint, config 등 파일 안의 주석 제거

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Summary by CodeRabbit

## Release Notes

### New Feature
- Added `CommentPage` and integrated it into the router with a new path `PATH.COMMENT`.
- Implemented comment-related components including `CommentActions`, `CommentInput`, `CommentItem`, and `CommentSection`.
- Introduced Firestore-based functions for creating, reading, updating, and deleting comments.

### Refactor
- Replaced `Comment` type with `CommentModel` and updated related data structures.
- Modified `fetchPosts`, `fetchUsers`, and `fetchComments` APIs to align with new data models.

### Chore
- Updated `.vscode/launch.json` to use Firefox for debugging.
- Adjusted ESLint ignore patterns in `functions/.eslintrc.cjs`.

These changes enhance the commenting functionality and improve overall code maintainability.

Copy link

github-actions bot commented Sep 6, 2024

Image description CodeRabbit

Walkthrough

이번 변경 사항은 댓글 기능을 추가하고, Firestore와의 통합을 통해 데이터 조작을 구현하며, 새로운 페이지와 컴포넌트를 도입하여 사용자 인터페이스를 확장하는 데 중점을 두었습니다. 또한 디버그 설정과 ESLint 무시 패턴이 업데이트되었습니다.

Changes

파일 그룹 요약
.vscode/launch.json 디버그 구성에서 브라우저 유형을 Firefox로 변경하고 pathMappings를 추가하여 소스 매핑 수정
functions/.eslintrc.cjs ignorePatterns 배열에 대한 주석 제거
src/App.tsx, src/constants/path.ts, src/pages/Comment.tsx CommentPage를 import하고 PATH.COMMENT에 대한 route 추가
src/api/fetchComment.ts, src/api/fetchPosts.ts, src/api/fetchUsers.ts, src/types/comment.ts, src/types/post.ts Firebase Firestore를 사용하여 댓글 관련 함수들 추가 및 업데이트
src/components/comment/*.tsx 댓글 관련 컴포넌트 추가 및 수정

🐇 "코드가 바뀌어 새로워졌네,
댓글 기능이 빛나게 됐네.
Firestore와 손잡고,
더 나은 앱으로 거듭났네." 🌟


Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

@nakyeonko3 nakyeonko3 changed the title Enhancement/comment-84 [enhancement] 댓글 페이지 생성 Sep 6, 2024
Copy link

github-actions bot commented Sep 6, 2024

Visit the preview URL for this PR (updated for commit 1567c99):

https://toy-project-2-team-4-pizza--preview-cuf8b9eo.web.app

(expires Sat, 07 Sep 2024 10:33:39 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 70fe82c823fc0ae3a244966b629012f11f284e3b

@love1ace
Copy link

love1ace commented Sep 6, 2024

오 좋네요 승인합니다

@HSjjs98
Copy link
Contributor

HSjjs98 commented Sep 6, 2024

오 좋네요 승인합니다

누구세요?

- fetchPosts.ts 에서 getCommentsCount 삭제
- Post.tsx 에서 댓글이 증가 했을 때 `댓글 수`가 업데이트 되도록 수정
@devdeun devdeun merged commit f47e057 into develop Sep 6, 2024
3 checks passed
@nakyeonko3 nakyeonko3 deleted the enhancement/comment-84 branch September 7, 2024 05:29
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.

4 participants