Skip to content

Conversation

@ChoiSeungWoo98
Copy link
Contributor

@ChoiSeungWoo98 ChoiSeungWoo98 commented Aug 4, 2025

기능 생성

  • db 조회 테스트 부분 작업 완료
  • 필수 값 핸들러 추가 및 상태 부분 추가
  • 사용자 pc에 드라이버가 설치 되어야 하는 sql server 제거 및 oracle 기본 버전으로 변경

기능 개선

  • exception이 발생한 경우 코드 메시지만 담도록 수정
  • 에러 발생 시 9999만 발생시키던 문제 해결

에러 수정

  • sqlite3 부분 값 가져오는 게 달라 안되던 부분 수정

스타일 변경

  • 객체를 만들고 넘기도록 수정

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 뭐가 다른건가요?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

간단한 리팩토링이랑 포맷팅 적용 같습니다

Copy link
Member

@nahyeongjin1 nahyeongjin1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이렇게 보니 서비스 별로 백의 자리 숫자에 차이를 두는 것도 좋은 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그 부분은 회의를 통해 어떤 기능이 몇번 대를 사용할지 정하면 좋을 거 같습니다!

@ChoiSeungWoo98 ChoiSeungWoo98 changed the title 디비 연결 부분 작업 전 개선 디비 커넥션 작업 Aug 6, 2025
Copy link
Contributor Author

@ChoiSeungWoo98 ChoiSeungWoo98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

새롭게 추가된 부분도 리뷰 부탁드립니다.
감사합니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그 부분은 회의를 통해 어떤 기능이 몇번 대를 사용할지 정하면 좋을 거 같습니다!

@nahyeongjin1 nahyeongjin1 self-requested a review August 6, 2025 11:09
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고 하셨습니다.
그러나 스키마쪽 폴더 아래 /driver는 괜찮은데 /user_db는 뭔가 사용자 디비같아서 저희 프로그램이 설치하는 db 같지가 않네요

Copy link
Member

@nahyeongjin1 nahyeongjin1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다.

@nahyeongjin1 nahyeongjin1 merged commit 118e144 into develop Aug 7, 2025
4 checks passed
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.

4 participants