Skip to content

Feedback #1

@davidtran

Description

@davidtran
  • 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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions