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

[Feat] 챗봇 기능 구현 #38

Merged
merged 14 commits into from
Jan 29, 2025
Merged

[Feat] 챗봇 기능 구현 #38

merged 14 commits into from
Jan 29, 2025

Conversation

chaen-ing
Copy link
Member

#️⃣연관된 이슈

#24

📝작업 내용

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

스크린샷 2025-01-27 오후 9 49 41 위와 같은 챗봇 기능을 구현하였습니다.
  • 대화 내역 조회 시 페이지 번호를 넘겨서 10개씩 페이징을 하도록 구현하였습니다.
  • 챗봇에게 메세지를 보내면 경제와 관련된 내용에 한해서 답변을 받고, 사용자와 챗봇의 답변을 모두 저장합니다.

💬리뷰 요구사항(선택)

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

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

@chaen-ing chaen-ing added the ✨ Feat 새로운 기능 추가 label Jan 27, 2025
@chaen-ing chaen-ing added this to the #2 - 챗봇 milestone Jan 27, 2025
@chaen-ing chaen-ing requested a review from chanmin-00 January 27, 2025 12:53
@chaen-ing chaen-ing self-assigned this Jan 27, 2025
@chaen-ing chaen-ing linked an issue Jan 27, 2025 that may be closed by this pull request
2 tasks
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.

기능 잘 구현하신 거 같아요 수고하셨습니다!

import com.ripple.BE.chatbot.dto.ChatListDTO;
import java.util.List;

public record ChatListResponse(List<ChatResponse> postList, int totalPage, int currentPage) {
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.

앗 그러네요 수정해서 올렸습니다ㅎㅎ

}

@Operation(summary = "대화 내역 초기화", description = "챗봇과의 대화 내역을 초기화합니다.")
@PostMapping("/clear")
Copy link
Member

Choose a reason for hiding this comment

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

@DeleteMapping으로 변경해도 좋을거 같네요

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.

수고하셨어요!

@chaen-ing chaen-ing merged commit a1efd9b into develop Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 챗봇 기능을 구현한다
2 participants