Feedback from CodersX

[nhim175]

Trước tiên cảm ơn bạn đã hoàn thành pet project. Sau đây là những góp ý có thể sẽ giúp cho project của bạn hoàn thiện hơn. Về mặt UI: đẹp, sáng sủa, responsive hoạt động tốt, có sử dụng thư viện UI (ant design)

• Thêm eslint và prettier
• Tên field không cùng chung convention, nên dùng chung 1 style (nếu đã camelCase thì tất cả camelCase)
• Khi cuối trang không còn item nữa thì vẫn liên tục fetch
• README không cần mô tả cách app hoạt động
• Có thể thêm 1 folder pages chứa các page component, thư mục components chỉ chứa các reusable components
• Reload page -> login lại từ đầu. Nên lưu token vào localStorage
• Nên set baseUrl cho axios để khỏi lặp lại API host https://github.com/minhthao56/App-Exchange-Old-Goods/blob/master/src/components/Reply.js#L35
• Nên cho API host vào .env

UI issues:

• Nút login bị disable khi email và password đã được nhập
• Khi mới vào trang lần đầu ấn vào ô input ở hình dưới thì báo lỗi. Nút sign out hiển thị mặc dù chưa sign in
• Nếu là product của mình thì nên ẩn nút Exchange (chỉ hiển thị những gì có ý nghĩa)
• Gõ comment ấn Enter -> reload lại page (nên e.preventDefault()), ấn nút gửi cũng reload lại page

[minhthao56]

Dạ cám ơn anh đã review ạ! Code e còn lõm lắm .Em sẽ sửa lại chỗ ghi nhắc nhở. Mà chỗ hàm [e.preventDefault()] mỗi lần submit e đều có sử dụng và sau khi comment e cho fatch lại data (gọi lại hàm fetch axios). Do codesandbox chạy k ổn định nên load lại page. Trên local vẫn ok anh. Còn phần token e chỉ lưu vào localStorage khi bấm remmeber me. Em cám ơn anh nhiều!