Skip to content

feat: 대시보드 및 사용자 월별일별 기능 구현#85

Merged
jihun4452 merged 19 commits into
devfrom
feat/dashboard
May 8, 2026
Merged

feat: 대시보드 및 사용자 월별일별 기능 구현#85
jihun4452 merged 19 commits into
devfrom
feat/dashboard

Conversation

@2jiyong

@2jiyong 2jiyong commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Ⅰ. PR 내용 설명 (Describe what this PR did)

  1. UserRole admin 추가
  2. JWT 발급 시 UserRole 추가
  3. 대시보드용 api 2개 추가 - 메인 페이지 조회 시 접속으로 간주

Ⅱ. 관련 이슈 (Does this pull request fix one issue?)

Ⅲ. 검증 방법 (Describe how to verify it)

Ⅳ. 리뷰 시 참고 사항 (Special notes for reviews)
admin 유저는 DB에서 직접 변경 필요

@jihun4452 jihun4452 changed the title Feat/dashboard feat: 대시보드 및 사용자 월별일별 기능 구현 May 8, 2026
@jihun4452

Copy link
Copy Markdown
Member
  1. @Modifying 없어서 Spring이 INSERT를 SELECT로 해석하려다 런타임에 터질 수 있어서 추가 했어여. clearAutomatically도 같이 붙여서 1차 캐시 stale 문제도 같이
  2. 옆에 DailyStatsQueryService는 Clock 주입받아 쓰는데 여기만 LocalDate.now() 직접 써서 테스트에서 날짜 고정이 안 되는 문제 잡았음요. 빈은 이미 있어서 필드만 추가했어
  3. native query 안에 'ADMIN' 문자열이 4번 박혀있어서 enum 이름 바뀌면 런타임에야 집계 이상한 걸 알게 되는 게 있어서 수정했어여 :adminRole 파라미터로 빼서 UserRole.ADMIN.name()으로 바인딩했어유
  4. of(User user, ...) 팩토리가 어디서도 안 쓰이고 있어서 정리,,, 저장은 native query로 userId만 넘기는데 엔티티는 @manytoone으로 User 객체 들고 있어서 구조 불일치 같아 보여서(내가 보았을땐..), userId 컬럼만 남기고 dead code 제거해서 정리했음요.

@jihun4452 jihun4452 merged commit aa546e8 into dev May 8, 2026
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.

2 participants