Skip to content

알림 관련 API 도메인을 수정 + 버그 픽스#323

Merged
pinocchio22 merged 4 commits intodevfrom
fix/#319-API-Domain
Apr 22, 2026
Merged

알림 관련 API 도메인을 수정 + 버그 픽스#323
pinocchio22 merged 4 commits intodevfrom
fix/#319-API-Domain

Conversation

@pinocchio22
Copy link
Copy Markdown
Contributor

📌 이슈

✅ 작업 사항

1. 공지사항 API 변경 대응

  • 메이플랜드 공식 사이트 구조 변경으로 기존 공지사항 API 동작 방식 수정이 필요했습니다.
  • 기존에는 datetime 기반 커서 페이징을 사용하고 있었으나, 사이트 구조 변경 이후 해당 방식으로 페이징이 불가능해져 id 기반 커서 페이징으로 변경했습니다.
  • 알람 관련 API 중 set-read API를 제외한 나머지 엔드포인트를 v1 → v2 도메인으로 변경했습니다.
  • 이에 따라 Domain → Repository → UseCase 흐름에서 cursor 타입 및 매핑 로직을 함께 수정했습니다.

2. 설정 화면 공지사항 / 패치노트 페이징 UI 이슈 수정

  • 페이징 시 기존 뷰를 제거하지 않고 새로운 뷰를 추가하는 방식으로 동작하고 있어 데이터가 중복되는 문제가 발생했습니다.
  • 기존 데이터가 유지된 상태에서 전체 데이터가 다시 추가되면서 다음과 같은 문제가 발생했습니다.
예시:
* 초기 로드: 20개
* 페이징 후 예상: 기존 20 + 신규 20 = 40개
* 실제 동작: 기존 20 + 전체 40 추가 = 총 60개
  • StackView 내부 기존 subview를 제거하지 않고 append 형태로 UI를 구성하고 있었던 것이 원인이었습니다.
  • 페이징 시마다 기존 뷰를 제거한 뒤 전체 데이터를 기준으로 다시 렌더링하도록 수정하여 완전한 덮어쓰기 방식으로 동작하도록 변경했습니다.

3. 종료된 이벤트 링크 이동 이슈

  • 종료된 이벤트 클릭 시 잘못된 페이지로 이동하는 현상이 확인되었습니다.
예시:
* 앱에서 [루디브리엄의 미로] 클릭
* 실제 이동: [만우절 대소동] 페이지로 이동
  • 해당 이슈는 재현되지 않는 상태이며, 서버 데이터 또는 링크 매핑 문제 가능성이 있어 추가 관찰이 필요합니다.
  • 현재 코드 수정은 진행하지 않고 상태 모니터링 예정입니다.

👀 ETC (추후 개발해야 할 것, 참고자료 등)

  • 업데이트 예정이므로 v 3.0.2 -> 3.0.3으로 수정
  • 팀원들 요청에 따라 앱 이름 MLS -> 메랜사 수정

set-read 제외한 알람관련 API 도메인 주소를 v1->v2로 수정
페이징 시 기존 데이터에 덮어쓰기가 아닌 추가 되고있던 이슈 해결
+ 진행중 이벤트 클릭 이슈는 현재 확인 불가능
+ 버전 3.0.0 및 앱 이름 변경
@pinocchio22 pinocchio22 requested a review from dongglehada April 22, 2026 06:38
@pinocchio22 pinocchio22 self-assigned this Apr 22, 2026
@pinocchio22 pinocchio22 added fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 chore 빌드 설정, 프로젝트 설정 등 “로직에 영향 없는” 변경 labels Apr 22, 2026
Copy link
Copy Markdown

@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

This pull request updates the alarm and notification system by transitioning from API v1 to v2, changing the pagination cursor from a date string to an integer ID, and adding an ID field to alarm response entities. Project configurations were updated to version 3.0.3, and the demo app's dependency injection was modified, though some registrations were commented out. Feedback includes a warning about potential runtime errors in the demo app due to missing dependency resolutions and a performance concern regarding the UI update logic in the announcement and patch note views.

Comment thread MLS/Presentation/MyPageFeature/MyPageFeatureDemo/AppDelegate.swift
Copy link
Copy Markdown
Member

@dongglehada dongglehada left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다!
특이사항은 없어보여서 승인처리 하겠습니다 :>

@pinocchio22 pinocchio22 merged commit 84d973c into dev Apr 22, 2026
2 checks passed
@pinocchio22 pinocchio22 deleted the fix/#319-API-Domain branch April 23, 2026 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 빌드 설정, 프로젝트 설정 등 “로직에 영향 없는” 변경 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

알림 관련 API 도메인을 수정합니다. 버그를 수정합니다.

2 participants