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.

Sign up for GitHub

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

Jump to bottom

[Feature/116] 챗봇 API 연결 #122

Merged
merged 5 commits into from
Jul 16, 2024
Merged

[Feature/116] 챗봇 API 연결 #122

merged 5 commits into from
Jul 16, 2024

Conversation

moir2407
Copy link
Member

Type of change

  • Feature : 새로운 기능 추가
  • Bug fix : 버그 수정
  • Refactor : 코드 리팩토링 작업
  • Document : 문서작업
  • Test : 테스트 코드 작성 및 테스트 작업
  • Style : 코드 스타일 및 포맷팅 작업
  • CI/CD : CI/CD 작업 수정
  • Chore : 패키지 매니저, 라이브러리 업데이트 등의 작업

PR Desciption

변경 사항 설명

  • ML 서버의 챗봇 API 연결 (client 모듈)
  • 챗봇 API 결과를 반환할 채팅 API 및 컨트롤러 테스트 작성 (core 모듈)

PR Log

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

새롭게 배운 것

  • Spring OpenFeign을 이용한 외부 API 호출

관련 이슈

asas6978 added 4 commits July 11, 2024 01:22
✨ Feat - ML 서버 간 챗봇 api 작성 (client 모듈)
5a18485 
<footer>
- 관련: #116
🔧 Chore - client-api 파일 수정
8766188 
<body>
- 챗봇 URL 환경변수 추가
- openfeign) 기존 readTimeout 설정 값으로 ML 서버 간 통신이 원활하지 않아 시간 증가

<footer>
- 관련: #116
✨ Feat - Chatting api 작성 (core 모듈)
c24a399 
<footer>
- 관련: #116
🧪 Test - ChattingController 테스트 코드 작성
8fa8130 
<footer>
- 관련: #116
@moir2407 moir2407 requested a review from destiny3912 July 11, 2024 03:36
@moir2407 moir2407 self-assigned this Jul 11, 2024
@moir2407 moir2407 requested a review from luke0408 as a code owner July 11, 2024 03:36
@github-actions github-actions bot added ✨ feature 🧪 test 테스트 관련 🔧 chore 설정, 빌드 및 기타 내용 labels Jul 11, 2024
luke0408
luke0408 reviewed Jul 11, 2024
View reviewed changes
Copy link
Member

@luke0408 luke0408 left a comment

Choose a reason for hiding this comment

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

👍 수고하셨습니다!

다만 우려되는 점은 Chatbot Client를 통해 api 호출시에 발생할 수 있는 에러에 대한 처리가 부족해 이후에 유지 보수시 어려움이 생길 수 있을 것 같다는 느낌을 받았는데, 따로 처리할 만한 예외 상황이 없는 것일까요?

clients/client-api/src/main/kotlin/io/say/better/client/chatbot/dto/ChatbotResponse.kt
data class Response(
val answer1: String = "",
val answer2: String = "",
val score: Int = 0,
Copy link
Member

Choose a reason for hiding this comment

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

❓ answer1, 2와 score의 의미가 무엇인가요?

@moir2407
Copy link
Member Author

👍 수고하셨습니다!

다만 우려되는 점은 Chatbot Client를 통해 api 호출시에 발생할 수 있는 에러에 대한 처리가 부족해 이후에 유지 보수시 어려움이 생길 수 있을 것 같다는 느낌을 받았는데, 따로 처리할 만한 예외 상황이 없는 것일까요?

api 호출 시 별도의 예외 상황이 없었고, 우선적으로 기능을 구현하는데 집중했기 때문에 별다른 처리를 하지 않았는데 다시 점검해보고 발생할만한 예외 상황이 있다면 추가해놓도록 하겠습니다..!!

@luke0408 luke0408 merged commit ac0a662 into master Jul 16, 2024
1 check passed
@luke0408 luke0408 deleted the feature/116 branch July 16, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luke0408 luke0408 luke0408 approved these changes

@destiny3912 destiny3912 Awaiting requested review from destiny3912

Assignees

@moir2407 moir2407

Labels
✨ feature 🧪 test 테스트 관련 🔧 chore 설정, 빌드 및 기타 내용
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] 챗봇 API
2 participants
@moir2407 @luke0408