Conversation
cho-in-sik
requested review from
2yunseong,
Superkid0714,
dle234 and
jskimcs
Summary of ChangesHello, 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은 사용자 회원가입 과정에 새로운 기능을 도입합니다. '기수' 필드를 필수 입력 항목으로 지정하여, 사용자가 이 정보를 제공하지 않고 회원가입하는 것을 방지합니다. 이전에는 이 필드가 비어있을 경우 null로 표시되었으나, 이제는 필수 입력 누락 시 즉각적인 피드백을 제공하여 데이터 무결성과 사용자 경험을 향상시킵니다. 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}
| if (!generation || generation.trim() === "") { | ||
| setWarning({ ...resetWarning, generation: true }); | ||
| return; | ||
| } |
There was a problem hiding this comment.
현재 기수 필드가 비어있는지만 검사하고 있습니다. 하지만 사용자가 'abc'와 같이 숫자가 아닌 값을 입력할 수도 있습니다. 이 경우 Number('abc')는 NaN이 되어 API 요청 시 문제가 발생할 수 있습니다. 정규식을 사용하여 숫자만 입력받도록 유효성 검사를 강화하는 것이 좋습니다. 이렇게 하면 빈 문자열과 숫자 아닌 문자열을 모두 한 번에 처리할 수 있습니다.
| if (!generation || generation.trim() === "") { | |
| setWarning({ ...resetWarning, generation: true }); | |
| return; | |
| } | |
| if (!/^\d+$/.test(generation)) { | |
| setWarning({ ...resetWarning, generation: true }); | |
| return; | |
| } |
| value={generation} | ||
| onChange={(e) => setForm({ name: "generation", value: e.target.value })} | ||
| isWrong={isWarning.generation} | ||
| wrongMessage={"기수를 입력해주세요."} |
There was a problem hiding this comment.
앞선 댓글에서 제안한 유효성 검사 강화에 따라, 에러 메시지도 사용자에게 더 명확한 정보를 제공하도록 수정하는 것이 좋습니다. '기수를 입력해주세요'라는 메시지 대신, 숫자만 입력해야 한다는 점을 알려주는 것이 사용자 경험에 도움이 됩니다.
| wrongMessage={"기수를 입력해주세요."} | |
| wrongMessage={"기수는 숫자만 입력 가능합니다."} |
2 participants
관련 이슈
작업 분류
PR을 통해 해결하려는 문제가 무엇인가요? 🚀
PR에서 핵심적으로 변경된 부분이 어떤 부분인가요? 👀
큰 변경사항 없습니다-!
기수 미입력시 null기로 표시가 되어 필수입력값으로 변경합니다.
핵심 변경사항 이외 추가적으로 변경된 사항이 있나요? ➕
백엔드도 로직 추가작업 진행했습니다
추가적으로, 리뷰어가 리뷰하며 알아야 할 정보가 있나요? 🙌
이런 부분을 신경써서 봐주셨으면 좋겠어요. 🙋🏻♂️
체크리스트 ✅
reviewers설정assignees설정label설정