Skip to content

Conversation

@park-daeun
Copy link
Contributor

[#5]

👩🏻‍💻 작업 내용

  • 풋살, 축구, 농구, 테니스 메뉴에 맞게 매치아이템을 보이도록 구현합니다.

image

📝 PR 포인트

➕ 기타

  • @Yunkeun 매치 조회 검색 부분 ?startDate=2022-09-12&endDate=2022-09-14&gender=female 쿼리 이런 식으로 BE에 쏘면 그에 맞게 응답오도록 해주실 수 있나요?

@park-daeun park-daeun requested a review from Yunkeun September 12, 2022 12:25
@park-daeun park-daeun self-assigned this Sep 12, 2022
Copy link
Member

@Yunkeun Yunkeun left a comment

Choose a reason for hiding this comment

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

?startDate=2022-09-12&endDate=2022-09-14&gender=female에 알맞은 응답 가능합니다.
여기에 추가로 sports 종목에 따른 응답도 주면 좋을 것 같아요~!
?sports=futsal&startDate=2022-09-12&endDate=2022-09-14&gender=female 이렇게요.

그리고 페이지네이션 관련은 백엔드에서 주는게 좋을까요? 아니면, 프론트에서 작업하는게 좋을까요? 일단 TODO로 남겨놓을까요?

@park-daeun
Copy link
Contributor Author

?startDate=2022-09-12&endDate=2022-09-14&gender=female에 알맞은 응답 가능합니다. 여기에 추가로 sports 종목에 따른 응답도 주면 좋을 것 같아요~! ?sports=futsal&startDate=2022-09-12&endDate=2022-09-14&gender=female 이렇게요.

sports 는 FE쪽에서 필터링거치도록 했는데 얘도 BE로 넘길까용?

그리고 페이지네이션 관련은 백엔드에서 주는게 좋을까요? 아니면, 프론트에서 작업하는게 좋을까요? 일단 TODO로 남겨놓을까요?

훔.. 플랩풋볼은 한 페이지에서 모든 데이터를 다 보여주는 것 같은데욥..
저희는 플랩풋볼보다 훨씬 적은 데이터지만 하루씩 보여주는 플랩풋볼에 비교해 기간 설정이 있어 애매하네요...!
일단 TODO로 두고 데이터가 많아지면 생각해볼까요..??!

Base automatically changed from feat/18_store_setting to develop September 13, 2022 10:39
@Yunkeun
Copy link
Member

Yunkeun commented Sep 16, 2022

?startDate=2022-09-12&endDate=2022-09-14&gender=female에 알맞은 응답 가능합니다. 여기에 추가로 sports 종목에 따른 응답도 주면 좋을 것 같아요~! ?sports=futsal&startDate=2022-09-12&endDate=2022-09-14&gender=female 이렇게요.

sports 는 FE쪽에서 필터링거치도록 했는데 얘도 BE로 넘길까용?

그리고 페이지네이션 관련은 백엔드에서 주는게 좋을까요? 아니면, 프론트에서 작업하는게 좋을까요? 일단 TODO로 남겨놓을까요?

훔.. 플랩풋볼은 한 페이지에서 모든 데이터를 다 보여주는 것 같은데욥.. 저희는 플랩풋볼보다 훨씬 적은 데이터지만 하루씩 보여주는 플랩풋볼에 비교해 기간 설정이 있어 애매하네요...! 일단 TODO로 두고 데이터가 많아지면 생각해볼까요..??!

sports 필터링은 FE쪽에서 하는거로 하시죠!
페이지네이션 관련은 TODO로 남겨놓는게 좋을 것 같습니다!!

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