search

vinaas / easyquiztest

A product of VinaAS Co., Ltd
0 stars 8 forks source link

Cài lại momentjs bằng npm_Tấn Hiệp #8

Closed hiepelpulga closed 7 years ago

hiepelpulga commented 7 years ago

[update_lần_1_Hiệp]: Nổi bật nút tạo mới, phân trang, STT tăng, format dd-mm-yyy

baotnq commented 7 years ago

Rất tốt dó @hiepelpulga , chúng ta bắt đầu sử dụng nhiều sức mạnh của github, gitlab rồi đó. Tiếp tục nghe.

@tungptvn review sơ Pull Request của @hiepelpulga rồi merge vô nghe. không cần review kĩ, vì anh nghĩ ổn ah.

tungptvn commented 7 years ago

@hiepelpulga good good, nhưng lần sau một số thư viện như momentjs thì em cài cài trực tiếp luôn nhé, ko cần tạo thành file moment.js trong common đâu, em vào thư mục easyquiztest/app/ rồi cài lại giúp anh nhé

cd app
npm i momentjs --save-dev

và dùng bằng cách import moment from 'moment' nhé, xong rồi tạo 1 lại merge request giúp anh.

baotnq commented 7 years ago

@hiepelpulga em phối hợp với anh @tungptvn theo sát luôn dự án này nghe. Tập trung mỗi dự án này cho anh, cái EIC để cho Hải và Đức phụ trách ah.

hiepelpulga commented 7 years ago

@baotnq Vậy em sẽ tập trung theo sát để hiểu dự án này lun nha anh.^^ sáng giờ em cứ rối rối nhảy qua nhảy lại EIC với EasyQuizTest.

hiepelpulga commented 7 years ago

@tungptvn Em đã nhận. Em sẽ hoàn thành sớm và báo anh ạ.

hiepelpulga commented 7 years ago

@tungptvn Em đã chỉnh sửa phần momentjs. Nhờ anh xem giúp em ạ. Em ngồi mò mà vẫn không tạo được thêm 1 cái pull request nữa.nên comentt vào đây lun.@@

link commit: https://github.com/vinaas/easyquiztest/pull/8/commits/a66d3cdf3569cb91ce197d5159173c18d394f17d

tungptvn commented 7 years ago

@hiepelpulga anh merge vô master rồi nhé

hiepelpulga commented 7 years ago

@tungptvn nhờ anh giúp em fix lỗi này.

tungptvn commented 7 years ago

@hiepelpulga em sua file create-table.js theo cái này nhé

hiepelpulga commented 7 years ago

@tungptvn sửa lại cũng sai nữa a.lỗi ở dòng if á a..e coi trên mạng thấy đoạn code giống a..mà copy lại cũng vẫn sai..@@ hình

hình

baotnq commented 7 years ago

nhìn câu lệnh ghi ...migrate models not attached to this datasource..., @hiepelpulga xem lại file datasource.js có đầy đủ các models trên không? Có khi nào bị xót đó.

Ngoài ra, thử xoá bớt các models mình vừa tạo ra, như Question, Quiztest, ... từng cái một rồi chạy lại xem trường hợp nào không bị lỗi, để từ đó dò ra nguyên nhân

tungptvn commented 7 years ago

@hiepelpulga trong loopback mỗi model (Question, Quiz, ...) sẽ được cấu hình tương ứng với datasource nào ở file model-config, lỗi của em là do các models chưa attach vào datasouce db