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.

Sign up for GitHub

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

Jump to bottom

[Fix] 채팅 관련 API V2 연결 #170

Merged
merged 19 commits into from
Jan 15, 2025
Merged

[Fix] 채팅 관련 API V2 연결 #170

merged 19 commits into from
Jan 15, 2025

Conversation

HelloWorldDogs223
Copy link
Contributor

@HelloWorldDogs223 HelloWorldDogs223 commented Jan 12, 2025 •
edited by yyypearl
Loading

연관 이슈

close #160


📁 작업 내용

  • 채팅 관련 api들을 v2 버전으로 연결하고 타입을 지정하였습니다.

HelloWorldDogs223 and others added 15 commits January 5, 2025 20:50
@HelloWorldDogs223
fix(#169): 채팅 api v2 연결
9103730
@HelloWorldDogs223
fix(#169) type 지정
57b8cab
@yyypearl
feat(#164): CategoryToggle 컴포넌트 제작
36d3761
@yyypearl
feat(#164): 겜구 약관 및 개인정보 보호 페이지 제작, 경로 설정
14cd37a
@yyypearl
refactor(#164): /policy 하위 페이지 params를 이용해 하나의 페이지로 통합
6c3bda4
@HelloWorldDogs223
Merge pull request #165 from Gamegoo-repo/feat/#164
c05827b 
[Feat] 이용 약관 및 개인정보 보호 페이지 제작
@yyypearl
fix(#164): useSearchParams 클라이언트 실행을 위한 Suspense 설정
bd55833
@yyypearl
Merge pull request #166 from Gamegoo-repo/feat/#164
aa46260 
[Fix] useSearchParams 클라이언트 실행을 위한 Suspense 설정
@yyypearl
fix(#164): Suspense 내 children 추가
0064aeb
@yyypearl
Merge pull request #167 from Gamegoo-repo/feat/#164
f367671 
[Fix] policy 페이지 Suspense 내 children 추가
@HelloWorldDogs223
fix(#169) 주소 변경
1cef6e9
@HelloWorldDogs223
Merge branch 'develop' of https://github.com/Gamegoo-repo/Gamegoo-front…
b55a3a9 
… into fix/#169
@HelloWorldDogs223
Merge branch 'server2' of https://github.com/Gamegoo-repo/Gamegoo-front…
326ffe9 
… into fix/#169
@HelloWorldDogs223
fix(#169) 채팅 api2 연결
55ccc8d
@HelloWorldDogs223
fix(#169): api param 수정
4193cfa
@HelloWorldDogs223 HelloWorldDogs223 self-assigned this Jan 12, 2025
@netlify Netlify
Copy link

netlify bot commented Jan 12, 2025 •
edited
Loading

✅ Deploy Preview for gamegoodev ready!

Name Link
🔨 Latest commit dc758cc
🔍 Latest deploy log https://app.netlify.com/sites/gamegoodev/deploys/6785e465fda59e0008d1b69e
😎 Deploy Preview https://deploy-preview-170--gamegoodev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@HelloWorldDogs223 HelloWorldDogs223 changed the base branch from develop to server2 January 12, 2025 11:28
yyypearl
yyypearl approved these changes Jan 15, 2025
View reviewed changes
src/api/chat.ts Outdated
}: ChatRoomBoardRequest): Promise<chatRoomBoardResponse> => {
try {
const response = await AuthAxios.get(
`http://13.124.213.255:8080/api/v2/chat/start/board/${boardId}`
Copy link
Member

Choose a reason for hiding this comment

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

용혁님! AuthAxios랑 Axios 모듈은 env에서 base url을 가져오기 때문에, 직접 base url을 다 넣을 필요 없이 엔드포인트만 넣으시고 env에서 NEXT_PUBLIC_BASE_URL 바꿔 사용하시면 됩니다🙌🏻

아래에도 이렇게 사용하신 부분이 많은데, 그럼 요청이 제대로 안 가지 않나요!?

@yyypearl
Copy link
Member

배포 에러 수정하시느라 고생 많으셨습니다ㅜㅎㅎ 머지할게요!

@yyypearl yyypearl merged commit b691334 into server2 Jan 15, 2025
4 checks passed
@yyypearl yyypearl added the 🔧fix 버그 수정 label Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yyypearl yyypearl yyypearl approved these changes

Assignees

@HelloWorldDogs223 HelloWorldDogs223

Labels
🔧fix 버그 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@HelloWorldDogs223 @yyypearl