-
Notifications
You must be signed in to change notification settings - Fork 0
feat: #91 시나리오 이름 중복 방지 처리 #95
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
Conversation
시나리오 이름 중복 시 토스트메시지 표출
Summary of ChangesHello @dev-domo, 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은 시나리오 이름 중복을 방지하는 기능을 구현하여 데이터 무결성을 강화하고 사용자 경험을 개선합니다. 시나리오 추가 및 수정 과정에서 이름 중복이 발생할 경우, 사용자에게 명확한 토스트 메시지를 제공하여 혼란을 줄이고 올바른 입력을 유도합니다. 이를 위해 관련 뷰 컨트롤러와 뷰 모델에 필요한 로직과 데이터 전달 체계를 추가했습니다. 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. 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
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
시나리오 추가 및 수정 시 이름이 중복되는 것을 방지하는 기능이 추가되었네요. 전반적으로 PR의 목적에 맞게 잘 구현되었습니다. SettingScenarioViewController에서 중복 검사 로직을 담당하는 함수의 이름이 기능과 다소 맞지 않아 혼동을 줄 수 있는 부분이 있어 수정을 제안했습니다. 해당 부분을 개선하면 코드의 가독성과 유지보수성이 더 향상될 것 같습니다. 자세한 내용은 아래 리뷰 코멘트를 확인해주세요.
BeforeGoing/Presentation/Feature/Scenario/ViewController/SettingScenarioViewController.swift
Outdated
Show resolved
Hide resolved
|


✅ PR 타입
반영 브랜치
refactor/#91-duplicateScenario -> dev
✨ 변경 사항
📂 관련 이슈
#91