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

[Feat] 피드 컴포넌트 마이그레이션 및 리팩토링 완료 #27

Merged
merged 6 commits into from
Nov 4, 2024

Conversation

juwon-code
Copy link
Collaborator

#️⃣연관된 이슈

See: #22

☑️ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • 코드 리팩토링
  • 파일 혹은 폴더 삭제
  • 파일 혹은 폴더명 수정

📝 작업 내용

피드 컴포넌트를 구현하고, 필요에 따라 프로필과 미디어 기능들을 수정했습니다.

✅ 피드백 반영사항

  • 정우님께서 작업해주신 코드 전반을 수정하였습니다.
  • 재형님께서 구현하신 랜덤 기능은 네이티브 쿼리로 랜덤 조회하는 것에서,
    페이징을 적용하여 랜덤 인덱스의 콘텐츠를 섞는 것으로 대체했습니다.

💬 리뷰 요구사항

대략적인 내용은 커밋 메시지를 참고해주시고, 세부 내용이 궁금하시다면
질문 주세요!

juwon-code and others added 3 commits November 5, 2024 01:29
피드 컴포넌트를 마이그레이션하고 잘못 또는 부족한 기능을 보완합니다.

- 피드 목록 랜덤 조회 API 등록.
- 피드-미디어 중간 엔티티로 관리.
- 피드 수정, 삭제시 권한 확인하도록 설정.

Related to: prgrms-be-devcourse#22
피드 도메인의 기능의 동작에 맞추어 미디어 및 프로필의 기능을 수정합니다.

- 미디어 <-> 프로필 간의 연관관계 설정.
- 불필요한 미디어 컨트롤러 삭제.
- 로그인한 회원의 정보를 제공하는 컴포넌트 작성.

Related to: prgrms-be-devcourse#22
`feature/feed-migration' -> `main`
피드 컴포넌트 마이그레이션 및 리팩토링 작업을 완료하고 `main` 브랜치로 병합합니다.

Related to: prgrms-be-devcourse#22
Copy link
Collaborator

@deveunhwa deveunhwa 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
Collaborator

@Preta3418 Preta3418 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

juwon-code and others added 3 commits November 5, 2024 02:40
채팅 조회에 슬라이싱을 적용하고, 피드 레포지토리의 ORDER BY가 누락된 조회 쿼리문을 수정했습니다.

Related to: prgrms-be-devcourse#28
`feature/chat-select-slicing' -> `main`
채팅 조회 슬라이싱 적용 완료하고 `main` 브랜치로 병합합니다.

Related to: prgrms-be-devcourse#28
@juwon-code juwon-code merged commit 9bc878a into prgrms-be-devcourse:main Nov 4, 2024
1 check failed
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.

3 participants