-
Notifications
You must be signed in to change notification settings - Fork 0
Feedback #1
Copy link
Copy link
Open
Description
davidtran
opened on Oct 27, 2020
Issue body actions
- Nếu chọn ngày bắt đầu lớn hơn ngày kết thúc => app sẽ bị lỗi. Cần check startDate và endDate không bị null
- Khi nhấn vào checkbox sẽ báo lỗi từ backend
- dòng 11 của todo.controller.js thì hàm requireQueryParams nên nhận vào 1 array để tránh viết lại nhiều lần. Tương tự như vậy cho 2 hàm requireParams còn lại
- Todo nên là một component riêng chứ không phải là hàm renderTodo
- Hạn chế dùng arrow function bên trong render.
- mapAction có thể là một object, không cần sử dụng bindActionCreators
- Ở hàm renderTodo thì param Todo là phải là dạng camelCase
- Component phải luôn viết hoa, đổi formCreate => FormCreate
- Chỗ render DELETE_TODO. Không cần phải tạo mảng mới […state.todos] bởi vì hàm filter luôn trả về mảng mới.
- Nên export trực tiếp function formateDate thay vì tạo class Utils.
- Tên thư mục nên viết thường: api, components, store, Utils
- Nên dùng async await thay vì .then
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels