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

[2.2.0] 쿠링봇 서버 연결 #221

Merged
merged 4 commits into from
Aug 22, 2024
Merged

Conversation

wonniiii
Copy link
Member

내용

  • 사용자 입력 텍스트 기반으로 자동 답변 생성 구현했습니다.
  • 다음 마지막 PR에 SwiftData와 로딩 이미지, 메인화면에 쿠링봇 네비게이션 추가해서 올리겠습니다.(아마 내일 예정..)

코드

  • Satellite 이용해서 연결하려고 했는데, SSE는 json으로 response를 받지 않아서 일단은 SSEClient라는 클래스를 따로 만들어서 구현했습니다.
  • 기존의 BotFeature와 BotView 연결했습니다.
  • 현재는 테스트 차원에서 채팅에 limit를 안걸고 계속 채팅할 수 있게 해놨습니다.

스크린샷

RPReplay_Final1723413429.mov

*[추가] SSEClient 추가 | *[수정] BotFeature View 연결
@wonniiii wonniiii added the 쿠링봇 쿠링봇 기능 label Aug 11, 2024
@wonniiii wonniiii requested review from lgvv and sunshiningsoo August 11, 2024 22:07
@wonniiii wonniiii self-assigned this Aug 11, 2024
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

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

작업 감사합니다!
궁금한 부분들 위주로 리뷰해봤어요~! 저도 같이 배워가겠습니다~~

@x-0o0
Copy link
Contributor

x-0o0 commented Aug 13, 2024

아직 코드는 안봤지만 영상을 보면 메세지 전송 후에도 메세지 입력창에 메세지가 남아있다가
응답이 오면 리셋되고 있어요.
메세지 전송할 때 입력창을 리셋하는게 자연스러워보이는데 어떤가요?

Copy link
Contributor

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

쓰레드 관리에 대해서 공부해 보시면 큰 도움이 될것 같아요.

메인쓰레드에서 불리는 호출 안에서 또 DispatchQueue.main.async 를 사용하면 동작의 호출시점이 어떻게 될지 한번 예상도 해보시구요!

@wonniiii
Copy link
Member Author

wonniiii commented Aug 13, 2024

아직 코드는 안봤지만 영상을 보면 메세지 전송 후에도 메세지 입력창에 메세지가 남아있다가 응답이 오면 리셋되고 있어요. 메세지 전송할 때 입력창을 리셋하는게 자연스러워보이는데 어떤가요?

앗 안그래도 이 부분 수정해서 다음 PR에 같이 올릴 예정이었습니다 ! 확인해주셔서 감사해요 ! : )

@x-0o0
Copy link
Contributor

x-0o0 commented Aug 14, 2024

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

✅ 패키지를 성공적으로 빌드했습니다.

@lgvv lgvv changed the title [2.1.2] 쿠링봇 서버 연결 [2.2.0] 쿠링봇 서버 연결 Aug 21, 2024
lgvv
lgvv previously approved these changes Aug 22, 2024
@wonniiii wonniiii dismissed lgvv’s stale review August 22, 2024 11:01

The merge-base changed after approval.

lgvv
lgvv previously approved these changes Aug 22, 2024
@wonniiii wonniiii dismissed lgvv’s stale review August 22, 2024 11:09

The merge-base changed after approval.

@lgvv
Copy link
Member

lgvv commented Aug 22, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label Aug 22, 2024
Copy link

🗿 머지 시도 실패. 승인 개수, 테스트 성공 여부 등 자동 머지 조건을 만족하는지 확인하세요.

@wonniiii wonniiii enabled auto-merge August 22, 2024 11:21
@wonniiii wonniiii disabled auto-merge August 22, 2024 11:21
@wonniiii wonniiii merged commit 369b209 into develop/2.0.0 Aug 22, 2024
@wonniiii wonniiii deleted the feature/kuringbot/SSE branch August 22, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 쿠링봇 쿠링봇 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants