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

feat(discover): adding filter to discover pages (#74) #82

Merged
merged 3 commits into from
Aug 7, 2022
Merged

Conversation

ctuanle
Copy link
Collaborator

@ctuanle ctuanle commented Aug 7, 2022

  • note: prefill filter with params from url

- note: prefill filter with params from url
@vercel
Copy link

vercel bot commented Aug 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
remix-movie ✅ Ready (Inspect) Visit Preview Aug 7, 2022 at 8:27PM (UTC)

@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@Khanhtran47
mấy hôm trước bận, đến hôm nay mới code được.
m check thử xem kết quả mấy cái filter có bị gì k, t thấy cứ sao sao

@Khanhtran47
Copy link
Owner

@Khanhtran47 mấy hôm trước bận, đến hôm nay mới code được. m check thử xem kết quả mấy cái filter có bị gì k, t thấy cứ sao sao

@ctuanle ừ t cũng thấy sai sai ý, t thử filter theo giống rứa bên tmdb thì ra kết quả khác

@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@Khanhtran47
#58 (comment)
#58 (comment)
#58 (comment)

@ctuanle à t hiểu rồi, bên popularity của movie sai chính tả, thiếu chữ t

t vừa sửa lại 1 tí, lỗi typo với sai chỗ hiện thị genre

@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@Khanhtran47

@ctuanle với có bug kiểu ni, khi mà filter xong sau qua route filter thì cái select nó có chọn genre đều trên value thì nó hiện là All, sau t chọn genre khác thì hấn bị lỗi

giờ chắc đc rồi

bên top rated, m cho thêm minimum user votes tầm hơn 200-300 cho chính xác

ừm để t thêm

Copy link
Owner

@Khanhtran47 Khanhtran47 left a comment

Choose a reason for hiding this comment

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

@ctuanle m thêm xong thì merge luôn nha

PandaCuteGIF

@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@Khanhtran47

@ctuanle m thêm xong thì merge luôn nha

okay

m cho thêm minimum user votes tầm hơn 200-300 cho chính xác

giờ hợp lí hơn rồi, nãy ra mấy film gì toàn 0 điểm với 10 điểm 😅

với cả cái sort_by t để mặc định là giảm giần hết trừ cái original_title ra
sau ni có làm filter nâng cao thì sửa sau cx đc

@Khanhtran47
Copy link
Owner

giờ hợp lí hơn rồi, nãy ra mấy film gì toàn 0 điểm với 10 điểm 😅

với cả cái sort_by t để mặc định là giảm giần hết trừ cái original_title ra sau ni có làm filter nâng cao thì sửa sau cx đc

@ctuanle ừ nãy t kiếm đc cả phim 18+ đồ nude 1 đống 🤣

@ctuanle ctuanle merged commit 5f8a7e0 into develop Aug 7, 2022
@ctuanle ctuanle deleted the feature/74 branch August 7, 2022 20:34
@Khanhtran47 Khanhtran47 linked an issue Aug 7, 2022 that may be closed by this pull request
@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@Khanhtran47 giờ muốn merge code từ develop qua master thì phải làm sao ? bữa push thẳng từ local lên đc giờ k đc nữa rồi

@Khanhtran47
Copy link
Owner

@ctuanle m checkout qua master rồi git merge develop rồi push

@ctuanle
Copy link
Collaborator Author

ctuanle commented Aug 7, 2022

@ctuanle m checkout qua master rồi git merge develop rồi push

@Khanhtran47 nãy làm vậy mà k đc
image

@Khanhtran47
Copy link
Owner

Khanhtran47 commented Aug 7, 2022

@ctuanle m checkout qua master rồi git merge develop rồi push

@Khanhtran47 nãy làm vậy mà k đc image

@ctuanle à qua hình như t mới thêm cái protect cho master thì phải, để t bỏ
để t merge luôn 😅

@Khanhtran47 Khanhtran47 linked an issue Aug 8, 2022 that may be closed by this pull request
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.

Discover page Filter component
2 participants