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

Discover page #74

Closed
Khanhtran47 opened this issue Aug 4, 2022 · 10 comments · Fixed by #82
Closed

Discover page #74

Khanhtran47 opened this issue Aug 4, 2022 · 10 comments · Fixed by #82
Assignees
Labels
design need to add or change design feature New feature or request

Comments

@Khanhtran47
Copy link
Owner

Khanhtran47 commented Aug 4, 2022

@ctuanle Filter movie and tv with discover api

https://developers.themoviedb.org/3/discover/movie-discover
https://developers.themoviedb.org/3/discover/tv-discover

@Khanhtran47 Khanhtran47 added feature New feature or request design need to add or change design labels Aug 4, 2022
@Khanhtran47
Copy link
Owner Author

Khanhtran47 commented Aug 4, 2022

TODO for filter component: add select year, user score, runtime (advanced)

@Khanhtran47
Copy link
Owner Author

@ctuanle m thử lm cái filter ni nha, t qua lm bên cái search

@ctuanle
Copy link
Collaborator

ctuanle commented Aug 5, 2022

@ctuanle m thử lm cái filter ni nha, t qua lm bên cái search

@Khanhtran47 gọi api á ?

@Khanhtran47
Copy link
Owner Author

Khanhtran47 commented Aug 5, 2022

@ctuanle m thử lm cái filter ni nha, t qua lm bên cái search

@Khanhtran47 gọi api á ?

@ctuanle ừ call api (thay api movie list hiện tại bằng api discover movie đồ rứa sau thay mấy cái query của link url fetch để filter), t nghĩ là xài useFetcher của remix đồ để call lại, mé hơi rắc rối thiệt -.-

@ctuanle
Copy link
Collaborator

ctuanle commented Aug 5, 2022

@ctuanle ừ call api (thay api movie list hiện tại bằng api discover movie đồ rứa sau thay mấy cái query của link url fetch để filter), t nghĩ là xài useFetcher của remix đồ để call lại, mé hơi rắc rối thiệt -.-

@Khanhtran47 okay để t thử 🤖

@Khanhtran47
Copy link
Owner Author

@ctuanle ừ call api (thay api movie list hiện tại bằng api discover movie đồ rứa sau thay mấy cái query của link url fetch để filter), t nghĩ là xài useFetcher của remix đồ để call lại, mé hơi rắc rối thiệt -.-

@Khanhtran47 okay để t thử 🤖

@ctuanle hoặc khi filter thì m chuyển route sang 1 trang index ứng với cái filter luôn, k bt đk

@Khanhtran47
Copy link
Owner Author

@ctuanle à cái select từ cái dropdown lấy value hơi ngu tý 😅
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set/values

@ctuanle
Copy link
Collaborator

ctuanle commented Aug 5, 2022

hoặc khi filter thì m chuyển route sang 1 trang index ứng với cái filter luôn, k bt đk

@Khanhtran47 trang index ở đây là gì nhỉ, như movies/discover?filter_query_bla hay movies/index

à cái select từ cái dropdown lấy value hơi ngu tý 😅

😅

@Khanhtran47
Copy link
Owner Author

hoặc khi filter thì m chuyển route sang 1 trang index ứng với cái filter luôn, k bt đk

@Khanhtran47 trang index ở đây là gì nhỉ, như movies/discover?filter_query_bla hay movies/index

à cái select từ cái dropdown lấy value hơi ngu tý 😅

😅

@ctuanle kiểu movies/discover?sort=...?genre=...?...=... dạng rứa nếu m chuyển route còn muốn giữ nguyên route thì xài useFetcher

@ctuanle
Copy link
Collaborator

ctuanle commented Aug 5, 2022

@Khanhtran47 oke

@Khanhtran47 Khanhtran47 removed their assignment Aug 7, 2022
ctuanle added a commit that referenced this issue Aug 7, 2022
- note: prefill filter with params from url
@Khanhtran47 Khanhtran47 linked a pull request Aug 7, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design need to add or change design feature New feature or request
Projects
None yet
2 participants