THẢO LUẬN

thg 3 25, 2020 10:27 SA

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

+2
Avatar
đã bình luận cho bài viết
thg 3 25, 2020 8:32 SA

Bài viết chất lượng

0
thg 3 25, 2020 8:25 SA

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?

0
thg 3 25, 2020 8:25 SA

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?

0
Avatar
đã bình luận câu trả lời trong câu hỏi
thg 3 25, 2020 7:40 SA

2.JPG

Nó báo lỗi này luôn 😦

0

Tại sao phải dùng nginx trong trường hợp này ạ !

0
Avatar
đã bình luận câu trả lời trong câu hỏi
thg 3 25, 2020 7:15 SA

thanbnk b.minh su dụng update ok rui

0
thg 3 25, 2020 6:08 SA

Em cảm ơn bác nha

0

oki thanks bạn

0
thg 3 25, 2020 4:51 SA

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

0

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) rp1.pngrp2.png

0
thg 3 25, 2020 4:29 SA

b dùng version bao nhiêu ạ

0

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?

0
thg 3 25, 2020 4:15 SA

url thì ở function toMail có định nghĩa đó bạn

0
thg 3 25, 2020 4:13 SA

để 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 😀

0

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 reset-password-2.png

0
Avatar
đã bình luận câu trả lời trong câu hỏi
thg 3 25, 2020 3:17 SA

@nguyenhuukhuyenudn cảm ơn bạn

0
thg 3 25, 2020 2:20 SA

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 =))

0
thg 3 24, 2020 5:49 CH

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 😂

+2
Viblo
Hãy đăng ký một tài khoản Viblo để nhận được nhiều bài viết thú vị hơn.
Đăng kí