Skip to content

Conversation

@BAEK0111
Copy link
Contributor

@BAEK0111 BAEK0111 commented Sep 4, 2025

#️⃣연관된 이슈

#38

📝작업 내용

채팅 api 규칙에 맞게 수정
사용자를 Principal Details에서 가져오도록 수정

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

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

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

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

비고 (Optional)

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

- chatting api에서 memberId 호출 부분 PD로 변경
- Store Entity 연관관계 수정
- StoreRepository class -> interface 수정
- 필요한 ErrorStatus 추가
- 채팅방 생성 api request 수정
- chatting api에서 memberId 호출 부분 PD로 변경
- Store Entity 연관관계 수정
- StoreRepository class -> interface 수정
- 필요한 ErrorStatus 추가
- 채팅방 생성 api request 수정
 - develop 머지
 - JwtAuthFilter.java 수정
 - git.ignore에 firebase/service-account.json 추가
- develop pull & 충돌 해결
 - JwtAuthFilter.java 수정
- 규칙에 맞게 Controller 수정
…urityutil-to-principaldetails' into refactor/#38-chatting-change-securityutil-to-principaldetails

# Conflicts:
#	.gitignore
#	src/main/java/com/assu/server/domain/chat/controller/ChatController.java
#	src/main/java/com/assu/server/domain/store/entity/Store.java
#	src/main/java/com/assu/server/domain/store/repository/StoreRepository.java
#	src/main/java/com/assu/server/global/apiPayload/code/status/ErrorStatus.java
@BAEK0111 BAEK0111 self-assigned this Sep 4, 2025
@BAEK0111 BAEK0111 added the ♻️ refactor Refactor code label Sep 4, 2025
Copy link
Contributor

@leesumin0526 leesumin0526 left a comment

Choose a reason for hiding this comment

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

굿좝

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.

고생하셨습니다!

- 규칙에 맞게 Controller 수정
- 프론트 연결 중 필요한 내용 추가
…into refactor/#38-chatting-change-securityutil-to-principaldetails

# Conflicts:
#	src/main/resources/application.yml
#	src/test/java/com/assu/server/ServerApplicationTests.java
…into refactor/#38-chatting-change-securityutil-to-principaldetails

# Conflicts:
#	src/main/java/com/assu/server/global/apiPayload/code/status/ErrorStatus.java
@BAEK0111 BAEK0111 merged commit 6d7d0aa into develop Sep 14, 2025
1 check passed
@2ghrms 2ghrms deleted the refactor/#38-chatting-change-securityutil-to-principaldetails branch September 17, 2025 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ refactor Refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants