Question Domain 확인된 api 개발 완료

RulLu16 commented 3 years ago

GET /question/search?tags=hello/string/kakao 처럼 해서 태그로 주면 해당 태그를 모두(and) 갖고 있는 문제만 반환됩니다. string 으로 어떤 구분자로 줄지는 프론트에게 한번 물어보면 좋을거같아요

[
    {
        "id": 2,
        "content": "hello world i am question2",
        "bookmark_count": 1,
        "username": "룰루",
        "email": "yeeunlee0520@gmail.com",
        "tagList": [
            {
                "tagTitle": "kakao"
            },
            {
                "tagTitle": "hello"
            },
            {
                "tagTitle": "string"
            }
        ]
    }
]

sorting도 곧 추가할게용

RulLu16 commented 3 years ago

current_date 관련된거는 상현님 pr 머지되어서 develop 갱신되면 수정할게요~!

RulLu16 commented 3 years ago

일단은 새 requestparam으로 sort=createDate or bookmarkCount 로 받아서 정렬하도록 수정했습니다. 그리고 tags에 아무값도 없으면 그냥 모든 문제 반환합니다.

이게 기본 sort 값을 지정하려 했더니 방법을 아직 못찾아서.. 좀더 찾아서 이후 변경하거나 해볼게용.

일단은 page or 무한스크롤 다 생각 안하고 전부 반환합니다. 기능 얼추 완성되면 무한스크롤로 되도록 변경할게요.

RulLu16 commented 3 years ago

hateoas // querydsl..

👍

헐 hateoas 지금껏 스펠링 잘못알고있었네요 ㅋㅋㅋㅋㅋㅋㅋㅋ

d36choi commented 3 years ago

무한스크롤 방식은 rest api 쪽에서 pagination 해서 던져줘야하는건지 몰겟네요 스크롤을 얼추 내리면 page 1 -> page 2 비동기적으로 추가호출 이런 식이려나.. 좀 찾아봐야겠네요

RulLu16 commented 3 years ago

https://galid1.tistory.com/786 https://wbluke.tistory.com/18

네네 찾아보니까, 저희는 기존 paging 방식 그대로 그냥 pageable로 하면 되는거같고, 프론트쪽에서 스크롤 끝을 감지해서 ajax로 저희에게 다음 페이지를 요청하면 될거같아요.

흠.. 이것까지 해보고 머지할까요 아니면 상현님도 바로 참고할겸 일단 머지하고 다시 페이징용 이슈 팔까요??

RulLu16 commented 3 years ago

앗 그리고 생각해봤는데, 사실 쿼리dsl 쓰는게 이 검색이랑 퀴즈밖에 없을거같아요. 거기다 가장 까다로운 로직이 퀴즈랑 요 검색쪽 같은데, 이걸 제가 죄다 해버리는건 좀 아닌거같아서 퀴즈 쪽 로직은 상현님이 해주셔도 좋을거같아요! 그러면 저는 그동안 북마크나 페이징쪽 적용해볼것 같습니당

d36choi commented 3 years ago

음 좋은 블로그네요 네 그러시죠~

2021년 8월 2일 (월) 오후 11:01, YeEun Lee @.***>님이 작성:

앗 그리고 생각해봤는데, 사실 쿼리dsl 쓰는게 이 검색이랑 퀴즈밖에 없을거같아요. 거기다 가장 까다로운 로직이 퀴즈랑 요 검색쪽 같은데, 이걸 제가 죄다 해버리는건 좀 아닌거같아서 퀴즈 쪽 로직은 상현님이 해주셔도 좋을거같아요! 그러면 저는 그동안 북마크나 페이징쪽 적용해볼것 같습니당

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/dnd-side-project/dnd-5th-10-backend/pull/25#issuecomment-891052066, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJVYF4QRRHAJI2GU2NEKZNDT22QLVANCNFSM5BL6RIVQ .

d36choi commented 3 years ago

https://galid1.tistory.com/786 https://wbluke.tistory.com/18

네네 찾아보니까, 저희는 기존 paging 방식 그대로 그냥 pageable로 하면 되는거같고, 프론트쪽에서 스크롤 끝을 감지해서 ajax로 저희에게 다음 페이지를 요청하면 될거같아요.

흠.. 이것까지 해보고 머지할까요 아니면 상현님도 바로 참고할겸 일단 머지하고 다시 페이징용 이슈 팔까요??

해보고 머지하실까요 페이징은 저도 따로할게요

RulLu16 commented 3 years ago

넵. 페이징까지 적용해서 머지할게요~

dnd-side-project / dnd-5th-10-backend

Question Domain 확인된 api 개발 완료 #25