Skip to content

Conversation

@Chasyuss
Copy link
Member

@Chasyuss Chasyuss commented Aug 3, 2025

🚀 풀 리퀘스트 제안

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 스타일링
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타

✈️ 관련 이슈

close #267

Summary by CodeRabbit

  • 신규 기능

    • 플로팅 버튼이 숨겨지는 경로에 '/mytodo/scrap/edu'와 '/mytodo/scrap/job'이 추가되었습니다.
  • 버그 수정

    • 타인의 투두리스트 페이지에서 로딩, 에러, 데이터 없음 상태의 UI가 개선되었습니다.
    • 에러 발생 시 메시지가 제대로 표시되지 않던 문제가 수정되었습니다.
  • 기능 개선

    • 플로팅 모달에서 카테고리 및 그룹 선택 없이 제목만으로 투두를 등록할 수 있도록 변경되었습니다.
    • 홈 배너의 버튼 문구와 이동 경로가 "직업 추천받기"에서 "직업 담으러 가기"로 변경되었습니다.
    • 회원가입 동의 화면에서 이미 가입된 이메일일 때 더 구체적인 안내 메시지가 표시됩니다.

@Chasyuss Chasyuss requested a review from cywin1018 August 3, 2025 15:59
@coderabbitai
Copy link

coderabbitai bot commented Aug 3, 2025

Walkthrough

이 변경사항은 FloatingModal 컴포넌트에서 카테고리 선택 및 관련 로직을 제거하고, 할 일 추가 시 제목만 제출하도록 수정되었습니다. 또한, 여러 UI 컴포넌트의 상태 처리, 에러 메시지, 버튼 노출 조건, 네비게이션 경로, 문구 등이 QA 피드백에 따라 조정되었습니다.

Changes

Cohort / File(s) Change Summary
FloatingModal 카테고리 및 제출 로직 단순화
src/common/modal/FloatingModal.tsx
카테고리 선택 및 관련 상태, 뮤테이션 로직 제거. 할 일 추가 시 제목만 제출하도록 변경. 불필요한 코드 및 주석 삭제.
ShowLayout의 플로팅 버튼 노출 경로 확장
src/outlet/ShowLayout.tsx
플로팅 버튼이 숨겨지는 경로에 /mytodo/scrap/edu, /mytodo/scrap/job 추가.
배너 네비게이션 및 문구 변경
src/pages/home/components/Banner.tsx
온보딩 이동 경로를 /onboard에서 /jobselect로 변경, 버튼 텍스트를 "직업 추천받기"에서 "직업 담으러 가기"로 변경.
OtherTodoListPage 로딩/에러/빈 상태 UI 개선
src/pages/otherTodoList/OtherTodoListPage.tsx
로딩, 에러, 빈 데이터 상태 시 flexbox로 중앙 정렬, 에러시 return 누락 수정, 메시지 스타일 개선.
회원가입 이메일 인증 실패 메시지 변경
src/pages/signup/components/SingupAgree.tsx
이메일 인증 실패 시 메시지를 "이미 가입된 이메일 주소입니다."로 변경.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant FloatingModal
    participant useAddTodoMutation

    User->>FloatingModal: 입력 후 제출 클릭
    FloatingModal->>useAddTodoMutation: 제목만 포함하여 할 일 추가 요청
    useAddTodoMutation-->>FloatingModal: 성공/실패 응답
    FloatingModal-->>User: 모달 닫기 또는 에러 표시
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Assessment against linked issues

Objective Addressed Explanation
QA 피드백에 따른 UI/UX, 상태 처리, 메시지, 버튼 노출 등 수정 (#267)

Possibly related PRs

  • [🐛BUG] QA수정 #281: FloatingModal의 카테고리 및 관련 로직 제거, 뮤테이션 변경 등 동일 파일 및 기능에 직접적으로 관련된 PR입니다.
  • [✨FEATURE] 플로팅 UI 변경 #238: FloatingModal의 카테고리 선택 및 제출 로직 단순화와 직접적으로 연관된 변경 PR입니다.
  • main 최신화 #197: FloatingModal에서 카테고리 및 잡 관련 로직을 추가/제거하는 상반된 방향의 변경 PR로, 중첩되는 코드 영역이 많아 밀접한 관련이 있습니다.

Suggested labels

🐛Bug

Suggested reviewers

  • cywin1018

Poem

🐇
QA 요청에 귀 쫑긋,
카테고리 훌훌 벗어던져
할 일은 오직 제목뿐,
버튼도 경로도 깔끔하게!
토끼는 오늘도 뛰며
버그를 쏙쏙 잡아내네.
🌱✨

Note

⚡️ Unit Test Generation is now available in beta!

Learn more here, or try it out under "Finishing Touches" below.


📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 78c3d25 and a95e388.

📒 Files selected for processing (5)
  • src/common/modal/FloatingModal.tsx (3 hunks)
  • src/outlet/ShowLayout.tsx (1 hunks)
  • src/pages/home/components/Banner.tsx (2 hunks)
  • src/pages/otherTodoList/OtherTodoListPage.tsx (1 hunks)
  • src/pages/signup/components/SingupAgree.tsx (1 hunks)
🧰 Additional context used
🧠 Learnings (6)
📓 Common learnings
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#89
File: src/common/modal/SelectModal.tsx:71-79
Timestamp: 2025-05-14T13:09:58.217Z
Learning: SelectModal 컴포넌트의 "담기" 버튼과 "사람인에서 자세히 보기" 버튼 기능은 추후 별도로 구현될 예정입니다.
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#148
File: src/pages/myTodo/tabs/TodoListPage.tsx:8-8
Timestamp: 2025-05-23T06:50:28.407Z
Learning: TodoListPage 컴포넌트에서는 현재 반응형 디자인을 구현하지 않고 고정 마진(mx-[120px])을 사용하기로 결정했습니다.
📚 Learning: banner.tsx의 반복되는 직종 정보 코드는 api 연결 시 변경될 예정이므로 임시로 작성된 코드입니다....
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#44
File: src/pages/home/components/Banner.tsx:58-70
Timestamp: 2025-05-06T05:11:59.910Z
Learning: Banner.tsx의 반복되는 직종 정보 코드는 API 연결 시 변경될 예정이므로 임시로 작성된 코드입니다.

Applied to files:

  • src/pages/home/components/Banner.tsx
  • src/pages/otherTodoList/OtherTodoListPage.tsx
📚 Learning: the text in `challengelist` array in `src/pages/home/components/tab/challengetab.tsx` is temporary p...
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#44
File: src/pages/home/components/tab/ChallengeTab.tsx:3-7
Timestamp: 2025-05-06T05:01:21.613Z
Learning: The text in `challengeList` array in `src/pages/home/components/tab/ChallengeTab.tsx` is temporary placeholder content that will be replaced or removed in a future update.

Applied to files:

  • src/pages/home/components/Banner.tsx
  • src/pages/otherTodoList/OtherTodoListPage.tsx
📚 Learning: '직업 목록 보러가기' 버튼의 onclick 핸들러는 해당 페이지가 구현된 후 추가될 예정이며, 현재는 type="submit"으로 설정되어 있어도 의도적으로 동작하지 않는 상태입...
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#61
File: src/common/modal/AddJobModal.tsx:38-43
Timestamp: 2025-05-12T05:59:51.634Z
Learning: '직업 목록 보러가기' 버튼의 onClick 핸들러는 해당 페이지가 구현된 후 추가될 예정이며, 현재는 type="submit"으로 설정되어 있어도 의도적으로 동작하지 않는 상태입니다.

Applied to files:

  • src/pages/home/components/Banner.tsx
📚 Learning: signupemailverify.tsx의 하드코딩된 이메일 주소 "123dodream@naver.com"는 임시적인 것이며, api 연결 시 동적 데이터로 교체될 예정입니다....
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#225
File: src/pages/signup/components/SignupEmailVerify.tsx:48-48
Timestamp: 2025-07-19T03:29:17.376Z
Learning: SignupEmailVerify.tsx의 하드코딩된 이메일 주소 "123Dodream@naver.com"는 임시적인 것이며, API 연결 시 동적 데이터로 교체될 예정입니다.

Applied to files:

  • src/pages/signup/components/SingupAgree.tsx
📚 Learning: todolistpage 컴포넌트에서는 현재 반응형 디자인을 구현하지 않고 고정 마진(mx-[120px])을 사용하기로 결정했습니다....
Learnt from: Chasyuss
PR: TEAM-DoDream/DoDream_FE#148
File: src/pages/myTodo/tabs/TodoListPage.tsx:8-8
Timestamp: 2025-05-23T06:50:28.407Z
Learning: TodoListPage 컴포넌트에서는 현재 반응형 디자인을 구현하지 않고 고정 마진(mx-[120px])을 사용하기로 결정했습니다.

Applied to files:

  • src/pages/otherTodoList/OtherTodoListPage.tsx
🧬 Code Graph Analysis (1)
src/common/modal/FloatingModal.tsx (1)
src/hook/todo/useAddTodoMutation.ts (1)
  • useAddTodoMutation (18-29)
🔇 Additional comments (8)
src/pages/home/components/Banner.tsx (1)

16-22: 네비게이션 경로와 텍스트가 일관성 있게 업데이트되었습니다.

버튼 텍스트와 네비게이션 경로가 모두 직업 선택 플로우에 맞게 수정되어 사용자 경험이 개선되었습니다.

src/pages/signup/components/SingupAgree.tsx (1)

53-55: 에러 메시지가 더 구체적으로 개선되었습니다.

일반적인 실패 메시지 대신 "이미 가입된 이메일 주소입니다"라는 구체적인 메시지로 변경되어 사용자가 문제를 정확히 파악할 수 있게 되었습니다.

src/outlet/ShowLayout.tsx (1)

16-17: 플로팅 버튼 숨김 처리가 적절히 추가되었습니다.

스크랩 상세 페이지에서 플로팅 버튼을 숨기도록 라우트가 추가되어 UI 일관성이 향상되었습니다.

src/pages/otherTodoList/OtherTodoListPage.tsx (3)

29-33: 로딩 상태 UI가 개선되었습니다.

로딩 스피너가 화면 중앙에 위치하도록 스타일링이 추가되어 사용자 경험이 향상되었습니다.


36-38: 에러 상태 렌더링 버그가 수정되었습니다.

누락되었던 return 문이 추가되어 에러 상태가 올바르게 렌더링됩니다.


40-45: 빈 데이터 상태 UI가 개선되었습니다.

중앙 정렬과 적절한 텍스트 색상이 적용되어 빈 데이터 상태의 가독성이 향상되었습니다.

src/common/modal/FloatingModal.tsx (2)

8-8: 할 일 추가 뮤테이션이 적절한 훅으로 변경되었습니다.

useFloatingSubmitMutation에서 useAddTodoMutation으로 변경되어 더 명확한 의미를 가지게 되었습니다.


32-51: 할 일 추가 로직이 단순화되었습니다.

카테고리 선택 로직이 제거되고 제목만으로 할 일을 추가할 수 있도록 변경되어 사용자 경험이 단순화되었습니다. 뮤테이션 페이로드도 필수 정보인 todoTitle만 포함하도록 최적화되었습니다.

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai generate unit tests to generate unit tests for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@Chasyuss Chasyuss merged commit 293147b into main Aug 3, 2025
1 check passed
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.

[🐛BUG] QA수정

3 participants