Skip to content

Conversation

@djlim2425
Copy link
Contributor

#️⃣연관된 이슈

close #103

📝작업 내용

MeetingService와 ChatGPTClient 구분

text를 추출하여 openApi를 활용하도록 변경

키워드를 받아 저장하는 로직 추가

tag관련 entity 추가

docx도 첨부가능하도록 추가 -> 추가 이슈로 생성

🔎코드 설명(스크린샷(선택))

코드에 대한 설명을 작성해주세요.

💬고민사항 및 리뷰 요구사항 (Optional)

고민사항 및 의견 받고 싶은 부분 있으면 적어두기

비고 (Optional)

참고했던 링크 등 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

@djlim2425 djlim2425 requested review from 2ghrms, Jinho622 and hknhj July 30, 2025 08:08
@djlim2425 djlim2425 self-assigned this Jul 30, 2025
@djlim2425 djlim2425 linked an issue Jul 30, 2025 that may be closed by this pull request
5 tasks
Copy link
Member

@2ghrms 2ghrms left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! ERD도 업데이트 부탁드려요

@hknhj
Copy link
Contributor

hknhj commented Jul 30, 2025

태그가 뭔지 설명해주실 수 있을까요?

@djlim2425
Copy link
Contributor Author

@hknhj 키워드라고 했어야했는데 태그로 해버렸네요. 수정하겠습니당

@djlim2425
Copy link
Contributor Author

keyword로 수정했습니다

@djlim2425 djlim2425 added the ✨ Feature 기능 개발 label Jul 30, 2025
Copy link
Contributor

@hknhj hknhj left a comment

Choose a reason for hiding this comment

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

keyword가 안건지를 요약했을 때 추출된 키워드일까요?
만약 맞다면 왜 meeting과 keyword를 다대다로 했는지 알 수 있을까요?
하나의 meeting에 안건지의 keyword가 여러개 있으니까 그냥 1대다 아닐까요?

import org.hibernate.annotations.DynamicUpdate;

@Entity
@Table(name = "meeting_tags")
Copy link
Contributor

Choose a reason for hiding this comment

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

테이블명 수정해야할 것 같습니다!

@Jinho622
Copy link
Contributor

고생하셨습니다! ERD에 추가된 테이블 작성부탁드립니다!

@djlim2425
Copy link
Contributor Author

@hknhj 그거 수정하려고 이슈 파놨습니다!

@djlim2425 djlim2425 merged commit 8ffba3a into dev Jul 31, 2025
1 check passed
@djlim2425 djlim2425 deleted the feat/#103-ai-meeting-agenda-service branch August 12, 2025 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: AI meeting 안건지 요약 로직 구분

5 participants