Refactor Code AM - Relesae 5 (8.4 -> 10.4)

havietduc91 commented 9 years ago

Refactor code

Sửa lại toàn bộ các phần deleted, hay change status, change executed ra 1 action riêng, ko để chung với action edit nữa
Sửa lại toàn layout cho admin_layout, preview_layout
- Thay đổi thư mục để file css, js
- Xem sét xóa hết những phần thừa js, xóa các file js, css thừa
- Cố gắng dùng toàn bộ style của bootstrap
- Ko lấy các thẻ meta, .. thừa
- Chia ra thành 3 layout riêng cho 3 kiểu user (shop, mall , backoffice)
Review lại toàn bộ các luồng xử lý các action
Review lại phần user login & logout
Review lại coding convention
Review lại toàn bộ các text trên header xem nếu giống nhau cho từng user thì mình sẽ để vào chung 1 file common

havietduc91 commented 9 years ago

Refactor View

Thay đổi header cho từng trang
Thay đổi toàn bộ các phần cho giống với bootstrap
- Button,
- a, link
- select option
- input
- Label
- Form => hozi form
Footer chuyển cho giống bootstrap
Sửa các phần login đang bị lỗi giao diện

havietduc91 commented 9 years ago

Thay thế toàn bộ các form thành form của bootstrap!

Announcement
Pickup
Mall
Timeline banner
Stamp event
Mall news
Shop news
Shop group
Coupon

havietduc91 commented 9 years ago

Bị lỗi ở phần mall news -link
Già soát lại toàn bộ các form, đang dùng lại 1 lần extend nữa, cái này là ko cần thiết
Test lại toàn bộ các màn hình, các màn hình sau khi login ....
Bị lỗi ở phần tự động lấy ngày tháng đúng từ js

havietduc91 commented 9 years ago

@@ -161,11 +161,6 @@ /* Global Styles ========================================================================= / / Form Elements */ input,select,textarea,button { font:12px Arial, Helvetica, sans-serif; vertical-align:middle; color:#000; } -input.input-text,select,textarea, .form-control {

border:1px solid #C6C5C5;
border-radius: 0; -} -input.input-text,textarea { padding:6px 12px; resize:none; -webkit-appearance: none; border-radius: 0;} select { padding:2px; moz-box-sizing:border-box; @@ -1588,13 +1583,6 @@

/* ======================================================================================= */ -#creart_password{
border: 1px solid #7f7f7f;
background-color: #e5e5e5;
cursor: pointer;
height: 25px;
padding: 5px; -} .image-list .image{ width: 48%; float: left; @@ -1636,4 +1624,10 @@ float: left; margin-left: 3%; width: 35%; +} +.select-custom{
height: 30px;
margin-right: 5px;
width: auto;
float: left; }

havietduc91 commented 9 years ago

Xóa và unpublic announcement trong trường hợp cả 2 mall và shop
Refactor riêng phần mall news

havietduc91 commented 9 years ago

Add lại file jquery-ui.css và js và test lại tất cả các phần có duration :+1:
Xem lại chưa refactor quản lý mall của backoffice :+1:
Refactor lại phần partial :+1:
Chưa refactor phần backoffice :+1:
Khi upload ảnh lên trên 3 ảnh thì bị đẩy xuống dưới :+1:
Các phần hiển thị danh sách mà có button ở cuối hàng đang to quá thì chuyển sang btn-xs :+1:
Xem sét refactor toàn bộ các button của mall và của shop thành btn loại vừa :+1:

havietduc91 commented 9 years ago

Bug

Thống kê sai số lần fav news của shop Thay

SELECT 'fav_news' as type, count(DISTINCT(user_metadatas.user_id)) as total, DATE_FORMAT(user_metadatas.created_at, '%Y/%m') as sortdate FROM shop_news INNER JOIN user_metadatas ON (shop_news.shop_id = user_metadatas.object_id) where shop_news.shop_id in (10, 13) group by MONTH( user_metadatas.created_at) Thành
SELECT 'fav_news' as type, count(DISTINCT(user_metadatas.user_id)) as total, DATE_FORMAT(user_metadatas.created_at, '%Y/%m') as sortdate FROM shop_news INNER JOIN user_metadatas ON (shop_news.news_id = user_metadatas.object_id) where shop_news.shop_id in (10, 13) group by MONTH( user_metadatas.created_at)

havietduc91 commented 9 years ago

A Hưng

Xem lại phần button ở các màn hình preview đang to nhỏ thất thường :+1:
Phần footer của các phần preview đang bị ẩn :+1:

Đức

Bug ở phần my-page coupon của shop khi chọn hiển thị theo ngày, theo tháng thì button bị tụt xuống dưới :+1:
Các input trong form bị sát nhau do form chưa phải là form horizontal
Check lại 1 lượt các file view xem có hard code phần URL ko? :+1:
Thay thế thư mục uploads hardcode thành define bằng config + move temp, fileCSV vào trong thư mục uploads :+1:
Đổi tên thư mục. Eg: stamp_event to stamp-event , .. :+1:
Move thư mục icon vào thư mục img :+1:
Sau khi thay lại font-size 14 cho các màn hình của device app & web :+1:
Màn hình chi tiết sử dụng mall ở backoffice bị bé hơn nhiều so với danh sách sử dụng mall :+1:
Tách css preview và mobile, xóa các class thừa của file CSS (80%)
Chuyển toàn bộ CSS có style ở table phiên bản device :+1:

havietduc91 commented 9 years ago

Bỏ các class của table :+1:

col-news-id
col-date
col-news-title
col-news-status
col-date
col-favourite
col-action-edit
col-shop-name
col-category-name
col-date-month

havietduc91 commented 9 years ago

Review lại 1 lượt các form của AM, với trình duyệt chrome (hay lỗi)

havietduc91 commented 9 years ago

Stamp event chưa có ảnh default?, click vào đang ra 4 ảnh và có 1 ảnh bị lỗi
Check lại phần change pass của mall và shop xem có bị sít vào nhau ko? :+1:

havietduc91 / laravel