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

Feat : 챗봇 기능을 수정합니다 (#24) #58

Merged
merged 4 commits into from
Feb 6, 2025
Merged

Feat : 챗봇 기능을 수정합니다 (#24) #58

merged 4 commits into from
Feb 6, 2025

Conversation

chaen-ing
Copy link
Member

#️⃣연관된 이슈

#24

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 챗봇 꿀팁 호출 시 DB 저장 및 응답 반환
  • 30일이 지난 대화 내역 삭제 기능 추가
  • 최신순으로 정렬하도록 변경

💬리뷰 요구사항(선택)

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

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@chaen-ing chaen-ing added the ✨ Feat 새로운 기능 추가 label Feb 5, 2025
@chaen-ing chaen-ing requested a review from chanmin-00 February 5, 2025 11:12
@chaen-ing chaen-ing self-assigned this Feb 5, 2025
chanmin-00
chanmin-00 approved these changes Feb 6, 2025
View reviewed changes
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 👍👍

src/main/resources/application.yml Outdated
@@ -8,6 +8,9 @@ spring:
ai:
openai:
api-key: ${OPENAI_API_KEY}
task:
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 Author

Choose a reason for hiding this comment

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

@ Scheduled가 잘안돼서 이것저것 테스트하다가 넣은건데 없어도 잘 돌아가네요!! 지웠습니다!!

@chaen-ing chaen-ing merged commit f05467d into develop Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chanmin-00 chanmin-00 chanmin-00 approved these changes

Assignees

@chaen-ing chaen-ing

Labels
✨ Feat 새로운 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chaen-ing @chanmin-00