Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. 관리자용 폴더 구조 정렬
• 기존 PR의 구조와 유사하게 admin 관련 파일 및 폴더를 정렬했습니다. 이를 통해 일관된 프로젝트 구조를 유지하고 가독성을 향상시켰습니다.
2. 의존성 주입 수정
• 현재 get_current_admin 함수가 미완성 상태여서 대신 get_current_active_user를 의존성으로 주입했습니다. 추후 get_current_admin이 완성되면 해당 의존성으로 교체할 예정입니다.
3. 스키마 작명 규칙 정립 필요
• routes/admin/response/loan_response 디렉토리 내 스키마 이름을 임의로 지정했습니다. 명확한 작명 규칙이 없어 일관성이 부족한 상태이므로, 추후 네이밍 컨벤션을 확립할 필요가 있습니다.
4. 데이터 무결성 문제 처리 고민
• loan 테이블에 존재하지 않는 user 또는 book을 참조하는 레코드가 있을 경우 오류가 발생합니다. 코드 단에서 이러한 무결성 문제를 감지하고 해당 레코드를 제외하거나 적절히 처리할지 고민 중입니다.
by GPT