Skip to content

#151 [Breaking Change] Admin 도메인 이관 및 관리자 UI 동기화#155

Merged
jucheonsu merged 5 commits intodevfrom
breaking/#151
Apr 11, 2026
Merged

#151 [Breaking Change] Admin 도메인 이관 및 관리자 UI 동기화#155
jucheonsu merged 5 commits intodevfrom
breaking/#151

Conversation

@jucheonsu
Copy link
Copy Markdown
Member

@jucheonsu jucheonsu commented Apr 10, 2026

#️⃣ 연관된 이슈

📌 공유 사항

  1. 관리자 백엔드 계층(Controller/DTO/Service)을 domain/admin으로 이동했습니다.
  2. 기존 battle/scenario/tag 내부 관리자 전용 클래스는 제거하여 책임을 분리했습니다.
  3. 관리자 폼(Battle/Quiz/Poll/Notice)의 입력 필드와 저장/조회 매핑을 동기화했습니다.
  4. 관리자 UI JS 동작(조회, 저장, 상태 토글) 및 템플릿 표시 문구를 정리했습니다.

✅ 체크리스트

  • Reviewer에 팀원들을 선택했나요?
  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 팀원들에게 PR 링크 공유를 했나요?

📸 스크린샷

관리자 목록/상세/생성/수정 화면

관리자 공지사항 작성/조회 화면

(실제 캡처 첨부)

💬 리뷰 요구사항

admin 도메인 경계 분리와 레거시 관리자 코드 제거 범위 중심으로 리뷰 부탁드립니다.

📝 작업 내용

[Breaking Change]

내용 파일
admin 도메인으로 계층 이관 domain/admin/**
알림/공지 관리 API 이관 domain/notification/**

[Remove]

내용 파일
레거시 관리자 클래스 제거 domain/battle/**/Admin*, domain/scenario/**/Admin*, domain/tag/dto/**

[Fix/Style]

내용 파일
관리자 JS 동작 보정 static/js/admin/**
관리자 템플릿 정리 templates/admin/**

@jucheonsu jucheonsu self-assigned this Apr 10, 2026
@jucheonsu jucheonsu added the 💥 Breaking Change (중대 변경) 커다란 API 변경이 있을 때 사용합니다. label Apr 10, 2026
@jucheonsu jucheonsu linked an issue Apr 10, 2026 that may be closed by this pull request
3 tasks
Copy link
Copy Markdown
Contributor

@si-zero si-zero left a comment

Choose a reason for hiding this comment

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

승인했습니다. 고생하셨어요. 머지 전에 서버에서 8081 포트 열렸는지 확인하고 연락드릴게요. 그때 머지해주세요!

@jucheonsu
Copy link
Copy Markdown
Member Author

승인했습니다. 고생하셨어요. 머지 전에 서버에서 8081 포트 열렸는지 확인하고 연락드릴게요. 그때 머지해주세요!

네 확인했습니다. 감사합니다!

@jucheonsu jucheonsu merged commit e522658 into dev Apr 11, 2026
@jucheonsu jucheonsu deleted the breaking/#151 branch April 11, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💥 Breaking Change (중대 변경) 커다란 API 변경이 있을 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

💥 Breaking Change: Admin 도메인 이관 및 관리자 UI 동기화

2 participants