최종발표용 v2 배포 #405
Merged
최종발표용 v2 배포 #405
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
ch8930
commented
Nov 17, 2024
ch8930
commented
- fcm 알림 기능 추가
- fcm기반 Notification 보내는 기능 개발 - gpt-api에게 STT 스크립트 내용 분석 요청하는 부분 개발 - Script Entity에 분석결과 컬럼 추가 및 gpt응답 결과 저장 후 fcm 보내도록 기능 - 녹음 기반 기능을 진행할 경우 STT 처리 후 분석결과 로직 추가함(+ 비동기로 처리함)
- 로그인시 fcm토큰 및 deviceId를 입력받아 새로 insert 혹은 update를 통해 갱신 - 로그인시 userEntity가 생성되면 FCM 토큰 처리에 대한 event 생성
- 유저 저장과 fcm 토큰 저장 api 분리하는 방식으로의 변경
- default themeId 조회하도록 변경
- 분석결과 조회할 수 있는 api 추가
- 분석결과 조회할 수 있는 api 추가
# Conflicts: # src/main/java/com/twentythree/peech/script/controller/ScriptController.java
- 예외처리 부분 보완
- fcm 시크릿 키 파일 생성 요류
- fcm 시크릿 키 파일 생성 요류
- 시크릿 키 등록 과정에서 이상하게 저장되어 파싱시에 문제 발생 시크릿값에는 인코딩해서 넣은 후 action에서 decode해서 넣도록 변경
- 테스트용 api
- @RequestBody 누락
- gpt 응답 결과 문제
- fcm notification 추가 - gpt 프롬프팅 수정
- 로그아웃 시 fcm 토큰 삭제 로직 - 비동기 스레드 풀 생성
Seungman-dev
approved these changes
Nov 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.