Conversation
Summary of ChangesHello @seseoju, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 일정 기능과 관련된 전반적인 QA 작업을 반영하며, 사용자 경험과 코드 품질을 향상시키는 데 중점을 둡니다. 일정 생성 및 수정 폼의 로직을 추상화하여 관리 효율성을 높였고, 데이터 포맷팅을 중앙 집중화하여 일관성을 확보했습니다. 또한, 지도 표시 기능을 개선하고 폼 유효성 검사를 강화하여 데이터 무결성을 높였습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
일정 기능 관련 QA 변경사항을 반영한 PR이네요. 전반적으로 코드 품질을 높이는 좋은 변경들이 많습니다. 특히, useScheduleFormState 훅을 분리하여 폼 관련 비즈니스 로직을 추상화하고, formatter.ts에 포매팅 함수를 통합하여 코드 중복을 줄인 점이 인상적입니다. 또한, 정적 지도 이미지 URL 생성 로직을 개선하여 다양한 화면 크기에 대응하고 커스텀 핀을 추가한 것도 좋은 개선입니다. 한 가지 개선점을 제안합니다. 일정 상세 페이지에서 일정이 꽉 찼을 경우 참여자가 취소할 수 없는 문제가 있어 수정을 제안했습니다. 자세한 내용은 개별 코멘트를 확인해주세요.
| </div> | ||
| </Show> | ||
| <Show when={!isCreator && isParticipating}> | ||
| <Show when={!isCreator && !isFull && isParticipating}> |
There was a problem hiding this comment.
일정이 꽉 찼을 경우, 이미 참여하고 있는 사용자가 참여를 취소할 수 없는 문제가 있습니다. !isFull 조건 때문에 취소하기 버튼이 보이지 않게 됩니다. 사용자는 일정의 참여 인원 상태와 관계없이 자신의 참여를 취소할 수 있어야 합니다.
| <Show when={!isCreator && !isFull && isParticipating}> | |
| <Show when={!isCreator && isParticipating}> |
1 participant
🛠️ 변경 사항
세부 변경 내용
useScheduleFormState훅을 분리했습니다.entity/schedule/lib/formatter.ts에 일정 기능에 필요한 포매팅(distance, pace, meetingAt)을 통합해 관련 페이지에 적용했습니다.🔍 관련 이슈
📸 스크린샷 / GIF (선택)
🔄 연관 작업