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

[PC-608] API 스키마 변경 대응 및 AI 요약 SSE 연결 #70

Merged
merged 16 commits into from
Feb 15, 2025

Conversation

tgyuuAn
Copy link
Member

@tgyuuAn tgyuuAn commented Feb 15, 2025

1. ⭐️ 변경된 내용

  • API 스키마 변경 대응
  • AI 요약 SSE 연결

2. 🖼️ 스크린샷(선택)

2025-02-15.10.37.41.mov

3. 💡 알게된 부분

  • SSE에 대해서 알게되었는데..

이거 알게된거 나중에 쓸게요 ㅠㅠㅠㅠㅠㅠ 머리아파

https://velog.io/@purang/SSE-Server-Sent-Events-Kotlin%EC%9C%BC%EB%A1%9C-%EA%B5%AC%ED%98%84%ED%95%B4%EB%B3%B4%EA%B8%B0

https://medium.com/@anugrahasb1997/implementing-server-sent-events-sse-in-android-with-okhttp-eventsource-226dc9b2599d

https://artdeveloper.tistory.com/38

@tgyuuAn tgyuuAn added UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 기능 ⚒️ 새로운 기능 구현 ⚒️ 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 새롭게 알게 됨 📖 프로젝트를 진행하며 새롭게 알게된 부분 📖 ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn labels Feb 15, 2025
@tgyuuAn tgyuuAn requested a review from sksowk156 February 15, 2025 13:40
@tgyuuAn tgyuuAn self-assigned this Feb 15, 2025
@sksowk156 sksowk156 added 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟 and removed 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 labels Feb 15, 2025
Copy link
Contributor

@sksowk156 sksowk156 left a comment

Choose a reason for hiding this comment

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

고생하셨어요!! ci 통과되면 바로 머지해주세요!!

@tgyuuAn tgyuuAn merged commit b4dca92 into develop Feb 15, 2025
1 check passed
@tgyuuAn tgyuuAn deleted the feature/tgyuu/PC-608 branch February 15, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn 기능 ⚒️ 새로운 기능 구현 ⚒️ 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟 새롭게 알게 됨 📖 프로젝트를 진행하며 새롭게 알게된 부분 📖
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants