Skip to content
This repository was archived by the owner on Nov 3, 2025. It is now read-only.

Feature/auth#47

Closed
chitsanupongkiw wants to merge 28 commits intomainfrom
feature/auth
Closed

Feature/auth#47
chitsanupongkiw wants to merge 28 commits intomainfrom
feature/auth

Conversation

@chitsanupongkiw
Copy link
Collaborator

รายละเอียดการเปลี่ยนแปลง

สรุปสิ่งที่แก้ไข เพิ่มเติม หรือลบออกใน Pull Request นี้
(สมมุติว่า ตัวอย่างเฉยๆนะเพื่อน เพิ่มระบบregister, ปรับปรุงการดักจับ error, แก้บั๊กถอนเงินไม่ถูกต้อง)

เช็คลิสต์ตัวอย่างเอาไปปรับแก้ตามฟังชันที่ตัวเองทำนะเพื่อนน นายเก่งอยู่แล้วระดับนี้

  • เพิ่ม/แก้ไขฟังก์ชัน ... stuff like that
  • แก้ไขไฟล์ ... something like that
  • อัปเดต comment ในโค้ด

การทดสอบ

อธิบายขั้นตอนการเทสในเครื่องของนายอะว่าฟังชันนายรันได้จริงป่าว ใส่ input ไรเข้าไปแล้วออกมายังไงไรงี้
เช่นแบบ รัน main_program.py แล้วเลือกเมนู Deposit, กรอก input เป็น string ให้มันผิด เพื่อดูว่าจับ error ได้จริงป่าว
แล้วก็แบบโชว์ด้วยว่า output ออกมายังไง ตรงไหม

input
//นายใส่ตรงนี้เลยว่าใส่ไรเข้าไป

output
// นายใส่ตรงนี้ว่าอะไรมันออกมาแบบไหน แล้วตรงไหมหรือไม่ตรง


เช็กลิสต์ก่อนขอรีวิว

  • โค้ดผ่านการรันโดยไม่มี error
  • ผ่าน test case ที่เกี่ยวข้อง
  • ไม่มีการแก้ไฟล์ที่ไม่เกียวข้อง
  • commit message ชัดเจน
  • ตรวจสอบแล้วว่าไม่ conflict กับ main

cReviewer

@pokpong-dev

เราจะเป็นตรวจโค้ดให้นายนะ ไม่ต้องร้อง เราเข้าใจ

@github-actions
Copy link

ไม่ผ่านว่ะ ไปนู่นไป อย่าทักมาอีกนะ รำคาญ

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants