Skip to content

[APM2-59] fix underline issue related to send merchant email to merchant when order status change to processing#233

Merged
nimitomise merged 4 commits intomasterfrom
disable-on-hold-order-email
Oct 21, 2021
Merged

[APM2-59] fix underline issue related to send merchant email to merchant when order status change to processing#233
nimitomise merged 4 commits intomasterfrom
disable-on-hold-order-email

Conversation

@nimitomise
Copy link
Copy Markdown
Contributor

  1. Objective
  • Send notification email to merchant once change order status to processing depending on payment method.

Related information:
Related PR(s): #232

  1. Description of change

Override Woocommerce filter for woocommerce_email_recipient_new_order which able to initiate with status as on-hold to not send order email to merchant.
Override woocommerce_order_status_processing to send notification email depend on payment method

  1. Quality assurance
  • no email send to merchant on complete checkout screen
  • found single email send to merchant when change order status to processing
  • new order email to customer both status as on-hold and processing

🔧 Environments:

Tested locally by pointing OMISE_API_URL to staging-omise

WooCommerce: v5.7.1
WordPress: v5.8
PHP version: 7.1

✏️ Details:

Make sure setting merchant email at /wp-admin/admin.php?page=wc-settings&tab=email&section=wc_email_new_order or /wp-admin/admin.php?page=wc-settings&tab=email to your valid email
Checkout cart with Paynow payment make sure customer email is valid and accessible
Go to customer mail box should still receive new order email
Go to merchant mail box should not receive new order email
Checkout cart with Credit Card and accessible
Go to customer mail box should still receive new order email
Go to merchant mail box should still receive new order email

  1. Impact of the change
    No

  2. Priority of change
    Normal

  3. Additional Notes
    No

@nimitomise nimitomise changed the title fix wrong variable name [APM2-59] fix underline issue related to send merchant email to merchant when order status change to processing Oct 20, 2021
@nimitomise nimitomise merged commit 6c59ae2 into master Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants