Cuuhomientrung / cuuhomientrung

Site hỗ trợ thông tin cứu hộ người dân miền Trung
https://cuuhomientrung.info
222 stars 48 forks source link

Hot fix bảng thống kê #233

Closed dangquangdon closed 3 years ago

dangquangdon commented 3 years ago

Hiện tại bảng thống kê bị hiển thị sai, tên Tỉnh và url không khớp nhau.

Problem

Độ dài của các list hodan_can_cuu, hodan_da_cuucuuho_by_tinh không tương xứng và các phần tử không hoàn toàn giống nhau, nên dù có được sort theo cùng thứ tự nhưng data lại khác nhau khiến việc update context bị sai.

không có id=2

hodan_da_cuu = [ { "tinh_id": 1, .... }, { "tinh_id": 3, .... } ]



Phải sửa lại logic để đảm bảo lấy ra đc đúng giá trị
dangquangdon commented 3 years ago

@DungDA bạn test lại giúp mình nhé, data ở local của mình có thể không đầy đủ nên có thể mình miss gì đó. 😞

DungDA commented 3 years ago

@dangquangdon mình cũng mới hotfix sửa 1 cái issue lúc lấy hộ dân ra, trong queryset bạn có lấy "tinh_id" mà nhiều khi. Tỉnh nó lại là None nên nó bị throw ra cái KeyError. Ok pull về test thử

dangquangdon commented 3 years ago

@DungDA mình mới forced push 1 tí tị, cơ bản là vẫn vậy, ko có j thay đổi, chỉ là thêm giá trị default = 0 cho mấy cái count thôi, bạn pull lại đc thì thử nhé.

Về cái lỗi hiện tại, trên production bạn thử xem cái dòng Hà Nội ấy. Căn bản mình thấy sao Hà Nội lại có lắm hộ cần cứu thế, ấn vào thì nhảy sang tỉnh khác. Một số trường hợp cũng bị vậy nên mình investigate lại và phát hiện ra lỗi ngu của mình ở PR trc. 😞

dangquangdon commented 3 years ago

Nhân tiện, bạn sửa lại text trong template cái chỗ này thành đội cứu hộ đã sẵn sàng thì cụ thể hơn, cho người dùng đỡ confuse, vì trong bảng thống kê hiện h là tổng đội cứu hộ (tất cả status)

image PS: số 40 trong hình là data trong local của mình thôi.