Skip to content

[16.0][IMP] purchase_request_approval_disbursement: auto-done purchase request on approval#681

Open
madara1150 wants to merge 1 commit into16.0from
16.0-imp-purchase_request_approval_disbursement-auto-done
Open

[16.0][IMP] purchase_request_approval_disbursement: auto-done purchase request on approval#681
madara1150 wants to merge 1 commit into16.0from
16.0-imp-purchase_request_approval_disbursement-auto-done

Conversation

@madara1150
Copy link
Copy Markdown
Collaborator

Summary

  • เมื่อ use_purchase_order = False และ purchase.request.approval เลื่อนสถานะเป็น approved จะปรับ purchase.request เป็น done ทันที
  • เมื่อ use_purchase_order = True จะปรับ purchase.request เป็น done หลังจากสร้าง Purchase Order ผ่าน approval_make_purchase_order แล้วเท่านั้น

Changes

  • Override button_approved() เพื่อเรียก request_id.button_done() เมื่อไม่ใช้ PO
  • แก้ไข approval_make_purchase_order() เพื่อเรียก request_id.button_done() หลังสร้าง PO

Test plan

  • ทดสอบ use_purchase_order = False: approve PA → ตรวจสอบว่า purchase.request state = done
  • ทดสอบ use_purchase_order = True: approve PA → ตรวจสอบว่า state ยังไม่เป็น done → กดสร้าง PO → ตรวจสอบว่า state = done

…est on approval

Auto-set purchase.request state to done based on use_purchase_order flag:
- When use_purchase_order=False: set done immediately upon approval
- When use_purchase_order=True: set done after PO is created via approval_make_purchase_order
@madara1150 madara1150 added the status/ready-for-review Status: Issue is up for grabs label Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/ready-for-review Status: Issue is up for grabs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant