Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 4, 2025

📄 요약(Summary)

csv 데이터 유효성 검증 및 회원 저장 로직 개선

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • Apache Tika를 통해 csv 또는 엑셀 파일의 실제 타입을 조회하여 유효성 검사
  • csv를 통한 회원등록 시, 개별 조회 및 개별 저장이 아닌 일괄 조회 및 일괄 저장 로직으로 수정하여 최적화

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@Sihun23

  • 테스트 코드에서 회원 등록 시 예외가 발생할 경우, 일부 회원만 저장되는지 검증하는 로직이 포함되어 있습니다.
  • 그러나 예외가 발생하면 전체 등록이 중단되어야 하는 것이 더 적절하다고 생각되어서, 해당 로직에 대한 기획 검토가 필요합니다.

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #306

@joowojr joowojr added 📝 docs 문서를 수정합니다 ✨ feature 구현·개선 사항에 관련된 내용입니다 labels Feb 4, 2025
@joowojr joowojr self-assigned this Feb 4, 2025
@joowojr joowojr changed the title CLAP-269 csv 데이터 유효성 검증 및 회원 저장 로직 개선 CLAP-269 csv 데이터 유효성 검증 추가 및 회원 저장 로직 개선 Feb 4, 2025
@joowojr joowojr merged commit a96fbb8 into develop Feb 4, 2025
1 check passed
@joowojr joowojr deleted the CLAP-269 branch February 4, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📝 docs 문서를 수정합니다 ✨ feature 구현·개선 사항에 관련된 내용입니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-269 엑셀 및 csv 파일 업로드에 유효성 검사 추가

2 participants