Issue - Githubissues

son-lt commented 9 months ago

@kullhy

[x] Detail product screen => getData
[x] Khi navigate từ màn hình Detail => Các button không navigate sang màn hình khác
[x] Thiếu padding dưới
[x] Cart => Detail Product screen => Thêm sản phẩm mới => Back về Cart => Data không reload https://github.com/lamdt-newwave/nws_huydq_ecommerce_flutter/assets/118146376/8eb8798e-3a32-49ad-ba96-ecee4adfba55
[x] Profile không chỉnh sửa được các trường thông tin
[x] Hạ kích thước của phần Total
[x] firebase_options.dart => service
[x] Không khai báo ở main
[x] Nên thêm dấu , ở giữa và format code
[x] app_navigator.dart đang có hàm forceSignIn (dùng để navigate) + với các hàm showDialog() => Nên tách ra
[x] app_svgs.dart => 'app_vectors.dart'
[x] Kiểm tra lại chính tả toàn app (Chỗ nào bị green underlines thì fix)
[x] api_path.dart => configs (nên đổi tên nữa)
[x] Tại sao lại có model trong bottom_bar
[x] dialog đang ở sai vị trí => Tham khảo lại base
[x] Đổi tên các components => widgets
[x] Không sử dụng StatefulWidget bừa bãi (Không dùng Consumer, Builder, Listener thì không cần dùng)
[ ] Tách nhỏ các Widget trong build thành các Widget con (tránh để build dài quá)
[x] Có thể xử lý cái đường dẫn trong hàm được không?
[x] genget_widget.dart + info_field.dart => Wrap trong StatelessWidget
[x] Nên để TextController ở bên page
[x] Add to cart vẫn cho phép bấm khi đang xử lý
[x] Không truyền cả cái cubit vào trong Widget như này
[x] Kiểm tra lại các đường dẫn được import => Absolute path
[x] Tên đặt chưa đúng và nên làm theo format của các cái còn lại
[x] Format code cho cả project
[x] Nếu không dùng đến thì không cần khai báo (Để ý xem giá trị mặc định các trường thuộc tính là gì)
[x] Không làm gì thì đừng làm :)
[x] Tạo màu trong app_color.dart
[x] cart_cubit.dart + detail_product_cubit.dart => https://flutterbyexample.com/lesson/assignment-operators
[x] Đoạn này bị vô nghĩa
[x] EdgeInsets.symmetric()
[x] BorderRadius.vertical()
[x] detail_product_cubit.dart
[x] Set giá trị mặc định cho GlobalData cũng được để không phải xử lý thêm như thế này
[x] Tìm cách để migrate data cũ (không có trường userId) để không xảy ra lỗi
[ ] Nên thêm static
[x] Trước khi quantity về 0 hiển thị dialog để xác nhận có xóa sản phẩm hay không? Nếu không, quay trở về 1 và đóng dialog
[x] #7
[x] Có 2 các xử lý: 1. Làm giống design, 2. Cho chữ chạy (Khuyến khích làm cách này) Expect: Actual:
[x] cart_page.dart => Đoạn này không cần thiết
[ ] Tại sao lại khai báo biến ở check_home_cubit.dart, main_cubit.dart, sign_up_cubit.dart (cần giải trình để xem cái nào giữ cái nào bỏ)
[x] Vẫn còn những đoạn như này ở trong project
[x] Trong DetailProduct đang cho quantity về 0 => Chặn
[ ] Những đoạn call API trong home_cubit.dart, login_cubit.dart, main_cubit.dart, profile_cubit.dart, sign_up_cubit.dart nên cho vào api_client.dart
[x] Thêm URL vào app_images.dart
[x] Những hàm trà về kiểu dữ liệu mong muốn từ dữ liệu được chọn như ở dưới thì hãy thêm vào trong một file nằm trong thư mục utils. Ví dụ như trường hợp này có thể thêm vào file date_utils.dart
[x] on_boarding_cubit.dart => Bỏ async
[x] on_boarding_page.dart => Hãy xử lý bằng cubit chứ không phải dùng setState
[x] cart_page.dart và profile_page.dart => Điều kiện đang sai (Tham khảo lại base hoặc hỏi lại anh)
[ ] Kiểm tra lại có một số Widget đang được sử dụng nhiều lần => Tạo thành common widgets