[#313] Release/v3.0.0 변경 사항 병합 #316
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary of Changes
Hello @Hrepay, 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
- 리뷰 평가 항목 선택 사항 변경: 리뷰 작성 시 맛과 양에 대한 평가를 선택 사항으로 변경하고, 이에 맞춰 관련 데이터 모델 및 UI 로직을 업데이트했습니다.
- 지도 초기 카메라 위치 고정: 앱 실행 및 지도 필터 변경 시, 지도의 초기 카메라 위치가 항상 '숭실대학교'로 고정되도록 개선하여 사용자 경험의 일관성을 높였습니다.
- 위치 권한 문구 및 빌드 설정 최적화: 위치 정보 권한 요청 문구를 사용자 친화적으로 개선하고, 빌드 설정을 최적화하여 앱의 전반적인 품질을 향상시켰습니다.
- 불필요한 코드 제거: 파트너십 데이터 모델에서 불필요한 필드를 제거하고, 지도에서 마커 중앙 정렬을 위한 불필요한 로직을 삭제하여 코드베이스를 정리했습니다.
Using Gemini Code Assist
The 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 in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.
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
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
| if response.tasteRating == nil { | ||
| tasteStackView.isHidden = true | ||
| } else { | ||
| tasteStackView.isHidden = false | ||
| tasteRateView.rateNumberLabel.text = "\(response.tasteRating ?? 0)" | ||
| } | ||
|
|
||
| if response.amountRating == nil { | ||
| quantityStackView.isHidden = true | ||
| } else { | ||
| quantityStackView.isHidden = false | ||
| quantityRateView.rateNumberLabel.text = "\(response.amountRating ?? 0)" | ||
| } |
There was a problem hiding this comment.
맛과 양 점수 UI를 처리하는 로직에 두 가지 개선점이 있습니다.
- 코드 중복: 이 로직은 270-282 라인의
myPageDataBind메서드에서도 동일하게 반복됩니다. 별도의 private 헬퍼 메서드로 추출하여 중복을 제거하는 것이 좋습니다. - 로직 불일치:
ReviewRateViewCell에서는rating == nil || rating == 0.0조건으로 숨김 처리를 하는 반면, 여기서는rating == nil만 확인합니다. 0점도 유효한 평가 점수인지, 아니면 '평가 없음'으로 간주해야 하는지에 대한 정책을 통일해야 합니다. 앱 전체의 일관성을 위해 두 파일의 로직을 맞추는 것을 권장합니다.
There was a problem hiding this comment.
해당 코드는 리뷰 v2에서 삭제될 예정이고, 별점 0점은 부여가 불가능해서 괜찮을 것 같습니다.
| let ssuLatitude = 37.49517278813046 | ||
| let ssuLongitude = 126.95661313346206 |
There was a problem hiding this comment.
숭실대학교의 위도, 경도와 같은 매직 넘버를 직접 사용하기보다 상수로 정의하여 사용하는 것이 좋습니다. 이렇게 하면 코드의 가독성이 향상되고, 나중에 위치를 변경해야 할 때 한 곳만 수정하면 되므로 유지보수가 용이해집니다. 파일 상단이나 별도의 Constants 파일/구조체에 정의하는 것을 고려해보세요.
There was a problem hiding this comment.
이건 수정이 필요합니다.
이슈를 파놓는게 좋을 것 같아요..
EATSSU/App/Sources/Presentation/Review/View/SeeReview/ReviewRateViewCell.swift
Show resolved
Hide resolved
| if response.tasteRating == nil { | ||
| tasteStackView.isHidden = true | ||
| } else { | ||
| tasteStackView.isHidden = false | ||
| tasteRateView.rateNumberLabel.text = "\(response.tasteRating ?? 0)" | ||
| } | ||
|
|
||
| if response.amountRating == nil { | ||
| quantityStackView.isHidden = true | ||
| } else { | ||
| quantityStackView.isHidden = false | ||
| quantityRateView.rateNumberLabel.text = "\(response.amountRating ?? 0)" | ||
| } |
|
#314 |
지금 이미 해당 릴리즈 브랜치의 코드로 배포가 완료된 상태라서 순서를 지키는게 맞을 것 같습니다! |
#️⃣ 관련 이슈
Resolved #313
💡작업 내용
💬리뷰 요구사항(선택)