-
Notifications
You must be signed in to change notification settings - Fork 5
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
⚡️ perf: 피드 삽입 정렬 기능 제거 & 피드 뷰 구현 #311
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생 많으셨습니다! 👍
몇가지 추가적으로 이야기 해보고 싶은 것들을 남겨놨어요!
덕분에 TypeORM으로 뷰쓰는 방법도 알고 갑니다. 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
큰 피드백은 성윤님께서 다 해주셨네요.
ViewEntity 데코레이터에 대해서도 알아가고 무엇보다 View에 대해 다시 공부하게 된 계기가 되었네요.
고칠 게 많았는데 정말 수고 많으셨어요! (__)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
질문사항 확인 감사드립니다 :)
🔨 테스크
Issue
피드 삽입시 정렬의 필요성 여부
📋 작업 내용
📷 스크린 샷(선택 사항)
개선 후
뷰 테이블
변경된 ERD