-
Notifications
You must be signed in to change notification settings - Fork 0
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
feature: 게시물 목록(API) #9
Comments
게시물 목록 구현을 위해 요구 사항을 분석 하였는데 올바르게 분석이 되었는지 확인이 필요합니다. ※ 게시물 목록에 보여지는 필수 필드 ( content_id, type, title, content(최대 20자), hashtags, view_count, like_count, share_count, updated_at, created_at)
※ 게시물 목록 API에선 content 는 최대 20자 까지만 포함됩니다. |
페이지네이션은 어떤 페이지네이션을 사용하실 예정이신가요?? 제가 알고 있는 페이지네이션같은 경우 리미트 옾셋, 커서, 일반 이렇게 지원하는 것으로 알고있습니다! |
페이지네이션 기능은 제공해주신 공식 문서를 기본 참고 문서로 두고 |
넵 좋습니다! 나중에 결정하시면 회의 때 알려주시면 감사하겠습니다! |
두 가지 Pagination 기능을 두고 어떤 것을 활용할까 고민했는데 일단 두 기능 모두 쿼리 파라미터 값을 받고, REST_FRAMEWORK = { “PAGE_SIZE” : 10 } 과 같이 전역 설정을 통해 LimitOffsetPagination 의 경우는 PAGE_SIZE 지정이 없을 때,
이게 가능한 것 같더라고요. request 요청 시 아래와 같이 값을 줘야 하기 때문에
|
- url을 restful하게 적용할 수 있도록 수정 - postlist 변수명 -> posts로 수정 - q 객체 사용하여 변수 지정 - 불필요한 if문 제거 - 반복적인 if-elif-else문 가독성있도록 수정 - ordering 부분 코드 수정 - test코드 추가 작성
- optional에서 type부분 리스트로 수정 - 관련 없는 코드 삭제 - search_keyword값이 있을 경우, title, content 검색 가능하도록 코드 작성 - 함수 리턴값 설정
- CI 과정 중 posts/views.py에 173번째 줄에 ordering이라는 변수를 사용하지 않아 제거 했습니다 Related to #9
- tests라는 파일과 폴더가 이중으로 존재하여 사용하지 않는 파일을 제거했습니다 Related to #9
- username 필드가 추가가 안돼서 username을 추가해줬습니다. Related to #9
- 테스트 케이스에 Print문 존재 Related to #9
Feed에 나타나는 게시물 목록 API
쿼리 파라미터
**를 사용 가능합니다.content
는 최대 20자 까지만 포함됩니다.The text was updated successfully, but these errors were encountered: