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

Nabi-NO-JIRA-fix : 이미지 다중 업로드 에러 로직 제거 #50

Merged
merged 3 commits into from
Nov 20, 2023

Conversation

born-A
Copy link
Collaborator

@born-A born-A commented Nov 20, 2023

PR 확인 항목

PR 보내기전에 아래 항목들을 만족 하였는지 체크 해주세요

  • 모진혁 팀에서 정한 커밋 메시지 규칙과 일치하는가?
  • 모진혁 팀에서 정한 코딩 컨벤션과 일치하는가?

PR 종류

어떤 종류의 PR인지 아래 항목중에 체크 해주세요

  • 버그 수정
  • 기능 추가
  • 코드 스타일 변경 (formatting, local variables)
  • 리팩토링 (기능 변경 X)
  • 빌드 관련 수정
  • 문서 내용 수정
  • 테스트 추가
  • 그 외, 어떤 종류인지 기입 바람:

어떤 기능이 추가 되었나요?

Issue Number: N/A
batchUpdate 반환값이 update 된 행의 개수라 서버를 올릴때 에러 메시지를 뱉어내는 것으로 생각됩니다.
따라서 void 로 변경을 하였습니다.
batchUpdate를 수행하며 이미지 batch insert 과정에서 발생할 수 있는 예외는 DataAccessException로,
런타임 exception이라 굳이 따로 처리를 안해줘도 된다고 판단을 하였습니다.

기존에 있던 기능에 영향을 주나요?

  • 아니요

기타

@born-A born-A self-assigned this Nov 20, 2023
@born-A born-A merged commit 1ad1b36 into dev Nov 20, 2023
1 check passed
@born-A born-A deleted the NABI-252--yejin--fix--suggestion-available-cards branch December 19, 2023 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants