Bên bạn có config CI không. Nếu chỉ là lỗi code convention, thì cài CI sẽ check được. PR chưa pass CI thì mọi người không review
Thường mình thấy lỗi khó sửa nhất đến từ mindset:
Vội vàng: Khi fix comment xong thì vội gửi PR lên rồi nhờ người khác check. Trong khi chính mình mới là người cần phải self check đầu tiên.
Không cẩn thận: Selfreview qua loa, với tư tưởng cứ gửi bản thô lên, ae trong team/ leader mới là người soi ra lỗi. Có comment thì mình sửa :v
Không có tư tưởng cải tiến code của mình: Khi code thường không có tư tưởng cầu tiến để viết ra code xịn hơn. Thường tìm 1 giải pháp tạm thời, hoặc trong PR còn 1 số chỗ 'lấn cấn' vì code chưa tốt, nhưng cũng không tham khảo ý kiến của người khác. Vẫn tư tưởng có comment thì fix, không thì merge luôn.
Code có nhiều comment thì hay bực tức :v cho rằng leader/ thằng khác thù mình nên comment nhiều để pull mình không được merge =))
...
Những lỗi này muốn sửa phải mưa dầm thấm lâu, nói nhiều thì member mới tự ngộ ra được :v
trong app/Notifications/ResetPassword.php mình thấy chỉ sửa được 1 số nội dung như cái text của button bên trên vài vài thứ khác.
Còn những cái mình khoanh đỏ thì mình chưa biết ở chỗ nào.
Trong views của mình không có folder vendor, có phải là do phiên bản không bạn?
để custom lại cái content này thì bạn vào file app/Notifications/ResetPassword.php ở function toMail rồi tùy chỉnh, và vào resources/views/vendor/notifications/email.blade.php tùy chỉnh theo ý của bạn, bạn thử test xem sao
Cảm ơn bạn vì bài viết bổ ích.
Có 1 số điểm mình chưa khắc phục được mong bạn giúp đỡ.
Mình muốn custom cái content mail này thì sửa ở đâu?
Và mình chạy ở local xài npm, laravel mix khi build ra cổng là localhost:3000 thì phải làm sao để cái $url gửi đi sẽ là cổng hiện tại thay vì là 127.0.0.1:8000
cty bạn có quy định gì về xử lý vấn đề này k "sao mày cứ hay lặp lại những lỗi không đáng có"?
lặp lại triền miên, code convention có tool check rồi mà tới tay review vẫn hiện, nhắc thì bảo e sẽ chú ý hơn, sau lại mắc
THẢO LUẬN
Bên bạn có config CI không. Nếu chỉ là lỗi code convention, thì cài CI sẽ check được. PR chưa pass CI thì mọi người không review Thường mình thấy lỗi khó sửa nhất đến từ mindset:
Những lỗi này muốn sửa phải mưa dầm thấm lâu, nói nhiều thì member mới tự ngộ ra được :v
cảm ơn em đã đọc bài này nhé!
Bài viết chất lượng
Hi bạn! Bạn cho mình hỏi khi test api bằng Katalon thì có điều gì cần lưu ý không?
Hi bạn! Bạn cho mình hỏi khi test api bằng Katalon thì có điều gì cần lưu ý không?
Nó báo lỗi này luôn
Tại sao phải dùng nginx trong trường hợp này ạ !
thanbnk b.minh su dụng update ok rui
Em cảm ơn bác nha
oki thanks bạn
chắc là cần chạy publish tag cho nó rồi, b có thể đọc thêm tùy chỉnh template ở đây xem sao https://link.sun-asterisk.vn/4uX3rB
Mình dùng bản 5.8. Bên dưới là Hình cây thư mục laravel của mình (ảnh 2 là trong vendor)
b dùng version bao nhiêu ạ
trong app/Notifications/ResetPassword.php mình thấy chỉ sửa được 1 số nội dung như cái text của button bên trên vài vài thứ khác. Còn những cái mình khoanh đỏ thì mình chưa biết ở chỗ nào. Trong views của mình không có folder vendor, có phải là do phiên bản không bạn?
url
thì ở functiontoMail
có định nghĩa đó bạnđể custom lại cái content này thì bạn vào file
app/Notifications/ResetPassword.php
ở functiontoMail
rồi tùy chỉnh, và vàoresources/views/vendor/notifications/email.blade.php
tùy chỉnh theo ý của bạn, bạn thử test xem saoCảm ơn bạn vì bài viết bổ ích. Có 1 số điểm mình chưa khắc phục được mong bạn giúp đỡ. Mình muốn custom cái content mail này thì sửa ở đâu? Và mình chạy ở local xài npm, laravel mix khi build ra cổng là localhost:3000 thì phải làm sao để cái $url gửi đi sẽ là cổng hiện tại thay vì là 127.0.0.1:8000
@nguyenhuukhuyenudn cảm ơn bạn
mỗi lầ n 1 nháy lên bảng phong thần cuối tháng tổng kết team tổ chức ăn chè chẳng hạn =))
cty bạn có quy định gì về xử lý vấn đề này k "sao mày cứ hay lặp lại những lỗi không đáng có"? lặp lại triền miên, code convention có tool check rồi mà tới tay review vẫn hiện, nhắc thì bảo e sẽ chú ý hơn, sau lại mắc