[Review] Hà Công Linh

hoangphantich / eula.web16.1

Code repos. Homepage:

https://hoangphantich.github.io/webeula16.1/

1 stars 3 forks source link

[Review] Hà Công Linh #8

Open hoangphantich opened 7 years ago

hoangphantich commented 7 years ago

@haconglinh1990 review 2 bạn này nhé @IceNeoMax Phạm Duy Nguyễn|985982154|https://www.facebook.com/WhiteNn|||nguyenneopham @kenvandemar Nguyễn Duy Khánh|943134461|https://www.facebook.com/kenvandemar|||silvercoe_oct13

[x] Review, comment code.
[x] Rating 1-10 theo tiêu chí của bản thân.
[ ] Ping mentor when done.

hoangphantich commented 7 years ago

@haconglinh1990 nhớ review nhé. Thấy đánh finish 2 tash rồi mà chưa nhìn thấy comment gì cả ^^

haconglinh1990 commented 7 years ago

@IceNeoMax 8/10 (Nên có css nhìn đẹp hơn, hàm if, else trong function validateFormOnSubmit() ko nên trả về false hết, nên ném 1 cái thông báo khi nhấn nút )

haconglinh1990 commented 7 years ago

@kenvandemar 8/10 (Gender có 3 lựa chọn đều là Male. Hơn nữa mặc định không chọn, gender + national nên để 1 giá trị mặc định, 2 trường password để trống khi submit không thông báo gì )

hoangphantich commented 7 years ago

Please review Test02 also.