Skip to content

Conversation

@Soundbar91
Copy link
Collaborator

🔍 개요

  • 코인 어드민 API 수정했습니다.

🚀 주요 변경 내용

  • 어드민 계정 생성, 권한 수정 시 슬랙 알림 로직을 추가했습니다.
  • can_create_admin 권한을 삭제하고, super_admin으로 통합했습니다.
  • team_type를 받지 않고, KOIN으로 고정으로 설정했습니다.
  • 클라이언트의 요청으로 요청값 네이밍을 JsonProperty 어노테이션을 사용하여 수정했습니다.
  • 테스트 코드를 수정했습니다.

💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@github-actions github-actions bot added Team User 유저 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다. 버그 정상적으로 동작하지 않는 문제상황입니다. labels Jan 20, 2026
@github-actions
Copy link

github-actions bot commented Jan 20, 2026

Unit Test Results

673 tests   670 ✔️  1m 22s ⏱️
165 suites      3 💤
165 files        0

Results for commit 8a25d82.

♻️ This comment has been updated with latest results.

Comment on lines +56 to +65
// 레거시 필드

@NotNull
@Enumerated(EnumType.STRING)
@Column(name = "team_type", nullable = false)
private TeamType teamType;

@Column(name = "can_create_admin", columnDefinition = "TINYINT")
private boolean canCreateAdmin = false;

Copy link
Collaborator

Choose a reason for hiding this comment

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

레거시 남겨놓은 이유가 있을까요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

컬럼 drop이 지금 ci 과정에서 막히기도 하고, null값을 넣자하니 nullable이 아니기 때문에 수정해야하는데 nullable으로 수정할 바에 나중에 한번에 drop 하는게 편할 거 같아 유지했습니다.

@Soundbar91 Soundbar91 merged commit aa69654 into develop Jan 22, 2026
5 checks passed
@Soundbar91 Soundbar91 deleted the fix/2139-koin-manager-api branch January 22, 2026 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team User 유저 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다. 버그 정상적으로 동작하지 않는 문제상황입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[어드민] 코인 어드민 계정 API 수정

4 participants