final flexible admin filter #249
Conversation
xuanbao2008
force-pushed
the
customized-flexible-chain-dropdown
branch
2 times, most recently
from
cb56ddf
to
fbaaf41
Compare
xuanbao2008
force-pushed
the
customized-flexible-chain-dropdown
branch
from
fbaaf41
to
119a013
Compare
|
Theo mình thì nên làm cái này ở backend, django có cái library |
Mình đã thử cái select2 mặc định của django, mình gọi nó thông qua .select2 trên element nhưng ko được, đã ráng dùng cách đó trước khi customize lại theo cách dùng external select2 lib |
Vậy à 🤔 Chắc mình sẽ thử lại sau, mình nghĩ cái select2 này có thể dùng đc ở nhiều chỗ khác nữa. Với cả mình góp ý chút là lần sau bạn viết commit message và description PR cụ thể 1 chút nhé 😓 |
|
|
||
| var all_huyen_data = []; | ||
|
|
||
| $.get("/get_huyen_api", function(response) { |
There was a problem hiding this comment.
hình như cái path này mình đã remove or PR #238 này rồi (merged to develop)
|
|
||
| var all_xa_data = []; | ||
|
|
||
| $.get("/get_xa_api", function(response) { |
There was a problem hiding this comment.
hình như cái path này mình đã remove or PR #238 này rồi (merged to develop)
There was a problem hiding this comment.
Mình có từng thấy bạn sửa cái api đấy trong develop branch, để mình pull branch đấy về rồi sửa theo.
There was a problem hiding this comment.
Bạn ơi, mình test cái merged PR trong develop branch, nó ok lắm, bạn có thể merge nó vào master luôn được ko, mình cũng tìm ra cách để fix cái pr của mình rồi, giờ chỉ cần adapt theo cái api path của bạn là ổn @dangquangdon
There was a problem hiding this comment.
merge develop -> master thì mình nghĩ để nhóm quán lý các bạn ý merge. Bạn có thể đổi lại target branch của PR này từ master sang develop cũng đc
There was a problem hiding this comment.
@xuanbao2008 cần chỉnh lại target branch thành develop
Hiện tại, cái git flow của mình đang hơi bị dị. Hết tháng 11 mình nghĩ mình sẽ xóa hết các branch, để lại mỗi branch master thôi, rồi archive repo.
| var remainingChars = args.minimum - args.input.length; | ||
| var message = 'Hãy nhập nhiều hơn ' + remainingChars + ' ký tự'; | ||
| return message; | ||
| }, |
There was a problem hiding this comment.
Tại sao phải có inputTooLong và inputTooShort nhỉ? Nếu không cần thì nên bỏ đi để code đỡ phức tạp.
| maximumSelected: function (args) { | ||
| var message = 'Bạn chỉ được chọn ' + args.maximum + ' mục'; | ||
| return message; | ||
| }, |
There was a problem hiding this comment.
maximumSelected, cái này cũng vậy. Mình đâu có chỗ nào cho phép multi choice đâu đúng không? Nên bỏ các code thừa đi nếu có thể.
| return 'Đang tìm kiếm...'; | ||
| } | ||
| }; | ||
| }); |
There was a problem hiding this comment.
Nếu mình không nhầm thì select2 có hỗ trợ sẵn tiếng Việt.
There was a problem hiding this comment.
Mình đã fix dc select2 mặc định rồi,nên code về translation ko cần nữa. Mình clean cho gọn rồi sẽ commit lên
Mọi người thử pull về local test thử nhé, nếu ok thì merge thẳng vào master luôn