Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @dodaaaaam, 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은 예상 작업 완료 시간 계산 로직의 문제를 해결합니다. 작업 완료 시간을 "예상보다 일찍 완료된 시간"으로 계산하는 유틸리티 함수를 분, 시간, 일 단위로 더 정확하게 표시하도록 리팩토링하고, 이 정보를 표시하는 UI 컴포넌트를 업데이트하여 변경 사항을 반영합니다. 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
예상 작업 완료 시간 계산 로직을 수정하는 PR이네요. 전반적으로 getEarlyFinishedHours를 더 상세한 시간 정보를 제공하는 getEarlyFinishedTime으로 교체하고, 관련 UI를 업데이트하는 변경 사항들이 잘 반영되었습니다. 다만, 새로운 함수의 반환 값 처리에 있어 몇 가지 논리적 결함과 개선점이 보여 리뷰를 남깁니다. 조건부 렌더링 로직과 API 설계를 조금 더 명확하고 안전하게 수정하면 좋겠습니다.
| subcontent={ | ||
| earlyHours !== -1 | ||
| ? `예상 작업 시간보다 ${earlyHours} 빨리 완료되었어요!` | ||
| : undefined | ||
| } |
There was a problem hiding this comment.
현재 조건 earlyHours !== -1은 getEarlyFinishedTime 함수가 반환하는 문자열 "-1"을 올바르게 처리하지 못하여 버그를 유발할 수 있습니다. 예를 들어, 작업이 예상보다 늦게 끝나면 earlyHours는 문자열 "-1"이 되고, "-1" !== -1은 true이므로 "예상 작업 시간보다 -1 빨리 완료되었어요!"라는 잘못된 메시지가 표시됩니다. 또한, 유효하지 않은 날짜에 대해 빈 문자열("")이 반환될 때도 의도치 않은 메시지가 보일 수 있습니다. earlyHours가 유효한 시간 문자열일 때만 메시지를 표시하도록 조건을 수정해야 합니다.
| subcontent={ | |
| earlyHours !== -1 | |
| ? `예상 작업 시간보다 ${earlyHours} 빨리 완료되었어요!` | |
| : undefined | |
| } | |
| subcontent={ | |
| (earlyHours !== -1 && earlyHours !== "" && earlyHours !== "-1") | |
| ? `예상 작업 시간보다 ${earlyHours} 빨리 완료되었어요!` | |
| : undefined | |
| } |
| const diffMs = estimated - completed; | ||
|
|
||
| // 일찍 끝나지 않았으면 | ||
| if (diffMs <= 0) return "-1"; |
There was a problem hiding this comment.
조금 더 일관성 있고 사용하기 쉬운 API를 위해, 일찍 끝나지 않은 경우에 매직 스트링 "-1"을 반환하는 대신 빈 문자열("")을 반환하는 것을 고려해보세요. 이렇게 하면 이 함수를 사용하는 쪽에서 ""와 "-1"을 모두 확인해야 하는 번거로움을 줄일 수 있고, falsy 값 체크만으로 간단하게 로직을 처리할 수 있습니다.
| if (diffMs <= 0) return "-1"; | |
| if (diffMs <= 0) return ""; |
| @@ -48,7 +48,7 @@ export function buildProcessSteps({ | |||
| }; | |||
|
|
|||
| const earlyHours = workData.print | |||
There was a problem hiding this comment.
SHOULD: 함수명에 맞게 이것도 바꾸면 좋을 거 같아요!
MlNTYS
left a comment
MlNTYS
left a comment
There was a problem hiding this comment.
gemini에 좋아요 누른 comment 확인 부탁드려요! 어푸 드립니다!
3 participants
🔀 Pull Request Title
현상관리 예상 작업 완료 시간 계산 로직 수정
📌 PR 설명
이번 PR에서 어떤 작업을 했는지 요약해주세요.