Skip to content

Conversation

@caniro
Copy link
Contributor

@caniro caniro commented Jul 2, 2025

✨ 작업내용

  • 마이페이지 총 자산 항목 추가

⚠️ 특별사항

  • 서비스 기동 시 asset 메모리 캐시하도록 추가했습니다.
  • 같이 모여있는 기간이 끝났으므로 신속한 리뷰가 어려울 것 같아,
    Github 설정에서 dev 브랜치 PR Merge 시 리뷰 필수 옵션 제거했습니다.
    image
  • SonarQube 비활성화 (수동 실행 가능)

@caniro caniro requested review from 109an94, Junh-b and bongj9 July 2, 2025 09:49
@caniro caniro self-assigned this Jul 2, 2025
@caniro caniro requested a review from BHyeonKim July 2, 2025 09:50
@caniro caniro linked an issue Jul 2, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@Junh-b Junh-b left a comment

Choose a reason for hiding this comment

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

구현하신 내용 확인했습니다.
assertservice쪽에서 캐시로 현재가 저장해두는것 좋은거 같네요

@caniro
Copy link
Contributor Author

caniro commented Jul 2, 2025

추가 변경내용 : 아이콘 DB 초기화 이후 캐시하도록 순서만 변경했습니다.
인메모리에서는 참조형이라서 똑같을텐데, 추후 외부 캐시 적용 등 변경 시 이 흐름이 맞는 것 같아서요.

@caniro caniro merged commit 5ee1cf5 into dev Jul 2, 2025
@caniro caniro deleted the feat/mypage-wallets branch July 2, 2025 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BE] 마이페이지 자산 추가

4 participants