Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PC-383] 어드민 차단 데이터 조회 기능 #28

Merged

Conversation

Lujaec
Copy link
Member

@Lujaec Lujaec commented Jan 20, 2025

🔗 관련 이슈

PC-383

✨ 작업 내용

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

🎃 새롭게 알게된 사항

📋 참고 사항

  • 코드 리뷰 시 논의가 필요한 사항이나 배포 관련 주의 사항을 추가합니다.

@Lujaec Lujaec requested a review from devchlee12 January 20, 2025 16:22
@Lujaec Lujaec self-assigned this Jan 20, 2025
Copy link
Member

@devchlee12 devchlee12 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
스웨거 명세만 살짝 바꾸시면 될 것 같아요.


@Service
@RequiredArgsConstructor
public class BlockContactService {
Copy link
Member

Choose a reason for hiding this comment

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

이름 명확해서 좋은 것 같습니다👍

Comment on lines +42 to +43
@Operation(summary = "핸드폰 번호 차단", description = "핸드폰 번호 리스트를 전달받고, 전달받은 핸드폰 번호 차단을 수행합니다.", tags = {
"차단"})
Copy link
Member

Choose a reason for hiding this comment

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

여기 swagger 명세가 좀 이상한 것 같습니다.
차단 조회 아닌가요?

@Lujaec
Copy link
Member Author

Lujaec commented Jan 24, 2025

고생하셨습니다! 스웨거 명세만 살짝 바꾸시면 될 것 같아요.

헉 코드를 재사용하다 보니까 잘못된 부분을 인지하지 못했네요 ,, 수정하겠씁니다

[PC-379] 어드민 모듈 로그인 기능
…ember-status

[PC-382] 어드민 프로필 상태 변경 기능
@Lujaec Lujaec merged commit fc6567a into feature/PC-384-admin-read-report Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants