Skip to content

Conversation

@pinocchio22
Copy link
Contributor

@pinocchio22 pinocchio22 commented Dec 9, 2025

📌 이슈

✅ 작업 사항

  • 상세페이지 필터 수정 / 탭바 이슈 수정]
  • 북마크 개선(데이터 리로드 방식 변경)
  • 상세 페이지의 Int타입 데이터에 numberFormmat 적용
  • 상세 조건 페이지 이동 추가
  • 마이페이지 비회원 분기처리
  • 도감 리스트 갱신 문제 해결 / 몬스터 필터 수정
  • 리스트 - 상세 북마크 동기화 / 북마크 관련 코드 개선 필요
  • 로그인 여부 / 데이터 유무 여부에 따라 UI 변경
  • 알림권한 설정 위치 수정 / 북마크 비로그인/로그인 처리
  • AddFolderCell 터치영역 수정
  • 북마크 하러가기 버튼 선택시 도감 탭 인덱스 변경
  • 컬렉션에 북마크 추가 후 메인으로 이동
  • 비로그인 북마크 UI 탭 수정
  • 탭 클릭 이동 방향 수정
  • 닉네임 설정 오류 수정
  • 닉네임 textField clearButtonMode 수정
  • 리스트 편집 아이콘 수정
  • 레벨 필터 초기값 수정(0 -> 1)
  • 최근검색어탭 노출 조건 수정
  • 북마크 온보딩 페이지 노출
  • 디테일 온보딩 구현
  • 디테일 온보딩 조건 추가
  • 약관 페이지 생성
  • 오픈소스 라이선스 표기
  • DetailStackInfoView.reset() 메소드 추가
  • 마이페이지 이미지 radius 수정

@pinocchio22 pinocchio22 self-assigned this Dec 9, 2025
@pinocchio22 pinocchio22 added feat 새로운 기능을 추가 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 labels Dec 9, 2025
@gemini-code-assist
Copy link

Summary of Changes

Hello @pinocchio22, 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!

이번 풀 리퀘스트는 내부 테스트 피드백을 반영하여 사용자 경험과 애플리케이션 안정성을 전반적으로 향상시키는 데 중점을 두었습니다. 북마크 기능의 실시간 동기화, 도감 상세 정보의 가독성 및 탐색 기능 개선, 그리고 알림 및 로그인 관련 사용자 흐름의 최적화가 주요 변경 사항입니다. 또한, UI 컴포넌트의 재사용성을 높이고 코드 구조를 개선하여 향후 유지보수 편의성을 증대시켰습니다.

Highlights

  • 북마크 시스템 개선: 북마크 데이터 리로드 방식이 개선되었고, 리스트와 상세 페이지 간 북마크 상태 동기화가 실시간으로 이루어지도록 변경되었습니다. 로그인/비로그인 상태에 따른 북마크 처리 로직도 강화되었습니다.
  • 도감 상세 페이지 기능 강화: 아이템 가격, 몬스터 스탯 등 Int 타입 데이터에 NumberFormatter를 적용하여 가독성을 높였습니다. 몬스터, 아이템, 맵, NPC 등 상세 페이지의 필터링 로직이 DetailType을 사용하여 더욱 유연하게 변경되었고, 퀘스트 요구사항에서 관련 아이템/몬스터 상세 페이지로 바로 이동할 수 있는 기능이 추가되었습니다.
  • 알림 및 로그인 플로우 개선: 알림 권한 설정 위치가 수정되었고, 온보딩 알림 화면에 뒤로가기 버튼과 동적인 권한 확인 로직이 추가되었습니다. 마이페이지 메뉴 항목 중 일부는 로그인 여부에 따라 접근이 제한되며, UI가 사용자 인증 상태에 맞춰 동적으로 변경됩니다.
  • UI/UX 및 코드 구조 개선: AddFolderCell의 터치 영역이 수정되었고, 비어있는 데이터 상태를 일관되게 표시하기 위한 DataEmptyView 컴포넌트가 도입되었습니다. 모달 프레젠테이션 시 탭바를 선택적으로 숨길 수 있는 기능이 추가되었으며, DictionaryListReactor는 낙관적 UI 업데이트와 효율적인 데이터 페칭 로직으로 리팩토링되었습니다. 또한, DictionaryTabRegistry를 통해 도감 탭 제어 기능이 중앙화되었습니다.
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 by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

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 pull request 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

  1. 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.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

내부 테스트 피드백을 반영한 대규모 수정 PR이네요. 북마크 동기화, 로그인 상태 관리, 알림 권한 처리, 빈 화면 UI 개선 등 앱 전반에 걸쳐 많은 개선이 이루어졌습니다. 특히 북마크 기능의 옵티미스틱 업데이트(낙관적 업데이트) 적용과 performBatchUpdates를 사용한 리스트 뷰 성능 개선은 사용자 경험을 크게 향상시킬 것입니다. 또한 NotificationPermissionManager와 같이 로직을 중앙에서 관리하도록 리팩토링한 점도 코드 유지보수성 측면에서 훌륭합니다. 전반적으로 코드 품질이 크게 향상된 좋은 PR이라고 생각합니다. 몇 가지 추가 개선을 위한 제안 사항을 아래에 남깁니다.

@pinocchio22 pinocchio22 merged commit 7d6e2c1 into dev Dec 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 새로운 기능을 추가 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants