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

[refactor] 게시물 조회 관련 api 보완 (응답값 추가) #24

Merged
merged 6 commits into from
Jun 10, 2024

Conversation

khee2
Copy link
Member

@khee2 khee2 commented Jun 10, 2024

PULL REQUEST

🔻관련 이슈

🎋 작업중인 브랜치

  • feautre/add-location-to-posts

📝 작업 내용

  • post api 서비스 응답값에 isMine, PostStatus 추가
  • 본인이 작성한 글은 찜 생성 및 채팅방 생성시 에러 처리 발생

🏞 스크린샷(선택)

  • post api 응답값 형태
image
  • 충돌 예상 부분(RedisConfig.java)

스크린샷 2024-06-10 171454

💬리뷰 요구사항(선택)

  • RedisConfig.java에서 redisTemplate함수의 bean 이름을 redisTemplate에서 customRedisTemplate으로 변경했습니다.
  • 변경 이유: RedisAutoConfiguration과 저희가 정의한 RedisConfig 간의 충돌 방지를 위해

khee2 added 5 commits June 10, 2024 10:00
- 로그인한 사용자가 작성한 게시글인지 확인하는 boolean타입의 isMine
- 현재 게시글 상태를 반환하는 PostStatus타입의 postStatus
- postLikeRepository를 사용하지 않고, 이미 엔티티(post) 내에 있는 데이터를 활용하여 좋아요 여부를 판단
- 데이터베이스 호출을 줄여 성능 향상
- redisTemplate의 bean 이름을 redisTemplate에서 customRedisTemplate로 변경
@khee2 khee2 changed the title Feature/add location to posts [refactor] 게시물 조회 관련 api 보완 (응답값 추가) Jun 10, 2024
@khee2 khee2 self-assigned this Jun 10, 2024
@Yeon-chae Yeon-chae merged commit a31f1fb into main Jun 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] 게시물 조회 관련 api 보완 (응답값 추가)
2 participants