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

fix : 활동보고서 API 파일 null 체킹 안 되는 에러 해결 #42

Merged

Conversation

wonjunYou
Copy link
Collaborator

🔥 연관 이슈

  • close #41

🚀 작업 내용

  • 활동보고서 API 수정 중, 이미지 파일 업로드를 안 하였음에도 불구하고 null 체킹이 안 되어,
    s3 업로드 시도 및 파일명 없이 히스토리 테이블에 row가 생성되는 에러 해결

image

  • 이런 형태로 form/data 요청을 보낼 때 파일이 없으면 null이 넘어오는 줄 알았지만, 실제로는 "" 가 넘어오게 된다.
  • 활동보고서 API가 추가 작성 및 수정의 편의를 위해 request에 대한 validation을 전혀 안 하다 보니 이런 작은 에러가 종종 생겨나네요 ㅠ

@wonjunYou wonjunYou merged commit ca72add into develop Feb 2, 2024
1 check passed
@5uhwann 5uhwann deleted the 41-활동보고서-이미지-NULL-체킹-버그-해결 branch July 1, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant