Skip to content
New issue

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

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

Already on GitHub? # to your account

[TSK-51] refactor: 입학타입 상세 #88

Merged
merged 14 commits into from
Oct 31, 2024
Merged

Conversation

Hoya324
Copy link
Member

@Hoya324 Hoya324 commented Oct 31, 2024

✅ 작업 내용

  • admission 관련 비즈니스 로직의 네이밍을 일관성 있게 리팩토링
  • admission 관련 CRUD Controller, Service, Repository 구현
  • admission 관련 통합 테스트와 리포지토리 테스트 추가
  • Swagger 보안 설정으로 경로 암호화 적용
  • 포트 설정 리팩토링

🤔 고민했던 부분

  • 비즈니스 네이밍 일관성: admission 기능에서 사용되는 네이밍이 통일되지 않아 코드 가독성이 떨어질 수 있다는 고민이 있었습니다. 이를 해결하기 위해 네이밍을 일관되게 수정하여 가독성을 향상했습니다.
  • 테스트 커버리지 강화: AdmissionTypeDetail 기능에 대한 통합 테스트와 리포지토리 테스트를 추가해 주요 비즈니스 로직의 안정성을 확인할 수 있도록 했습니다.

🔊 도움이 필요한 부분

  • 네이밍 적합성 확인: admission 관련 비즈니스 네이밍 리팩토링이 일관성 있게 잘 적용되었는지 리뷰 부탁드립니다. 팀원들의 의견이 일관성 유지와 가독성 향상에 큰 도움이 될 것입니다.
  • 보안 설정 검토: Swagger 경로를 암호화하는 방식이 적절한지, 추가적으로 보완할 점이 있는지 확인 부탁드립니다.

@Hoya324 Hoya324 added 🧪테스트 test를 추가한 pr입니다. ✨기능 새로운 기능을 추가합니다. labels Oct 31, 2024
@Hoya324 Hoya324 self-assigned this Oct 31, 2024
Copy link

Copy link

Unit Test Results

121 tests   121 ✔️  21s ⏱️
  23 suites      0 💤
  23 files        0

Results for commit 849ff4b.

Copy link

@Hoya324 Hoya324 merged commit 5727964 into develop Oct 31, 2024
2 checks passed
@Hoya324 Hoya324 deleted the refactor/입학타입-상세 branch October 31, 2024 19:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
✨기능 새로운 기능을 추가합니다. 🧪테스트 test를 추가한 pr입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant