-
Notifications
You must be signed in to change notification settings - Fork 1
[#360] 위젯 UI 버그 수정 #361
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
[#360] 위젯 UI 버그 수정 #361
Conversation
- 메뉴 하나일 때는 +로 연결 - 메뉴가 여러개일 때는 +로 연결하고 여백주기
- 메뉴를 받는 [String]을 [[String]]으로 변경 - Normal, Small 위젯 로직 수정 - 예시 메뉴 변경
Summary of ChangesHello @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은 위젯 UI 버그를 해결하기 위해 메뉴 데이터 관리 방식을 개선합니다. 기존의 단일 배열 메뉴 구조를 2차원 배열로 변경하여, 메인 메뉴와 반찬 같은 연관된 항목들을 논리적으로 그룹화할 수 있게 되었습니다. 이로 인해 위젯에 메뉴가 더 명확하고 구조적으로 표시되며, 데이터 처리 로직의 일관성이 향상됩니다. Highlights
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은 위젯에서 메뉴 데이터를 표시하는 방식을 개선하기 위해 데이터 구조를 [String]에서 [[String]]으로 리팩토링하는 중요한 변경을 포함하고 있습니다. 전반적으로 코드 변경 사항이 명확하고 PR의 목적에 잘 부합합니다. 몇 가지 중복 코드 제거, 잠재적 런타임 오류 방지, 그리고 프로바이더 간의 로직 일관성 확보를 위한 제안 사항을 리뷰에 담았습니다. 확인 부탁드립니다.
EATSSU/Widget/Sources/Presentation/Views/SuccessView/NormalView/MediumNormalView.swift
Outdated
Show resolved
Hide resolved
EATSSU/Widget/Sources/Presentation/Views/SuccessView/NormalView/SmallNormalView.swift
Outdated
Show resolved
Hide resolved
Funital
left a comment
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.
2차원 배열로 하나의 그룹을 만드니까 보기 더 편해졌네요
#️⃣ 관련 이슈
Resolved #360
💡작업 내용
핵심 변경 사항
세부 업데이트 내역
위젯 뷰(View) 로직 개선
네트워크 및 기타
💬리뷰 요구사항(선택)