Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GlobalStep - 4.2 RC1]: On MAC(Safari), Refunded transactions fails to appear in Payments -> Transaction tab. #673

Closed
juliaamosova opened this issue May 20, 2020 · 7 comments

Comments

@juliaamosova
Copy link

The original issue was filed in the WooCommerce Core repository: woocommerce/woocommerce#26563.

Copy-pasting since I can't easily transfer it:

Environment:
MAC Mojave v10.14.6: Safari(v13.0.5)

Steps to Reproduce:

  1. Create a site using Ninja tool.
  2. Install all the plugins.
  3. Tap on "Run the Setup wizard".
  4. Complete all the introductory tabs i.e., Store details, Industry ,Product types, Business details and Theme.
  5. Place an order using the site.
  6. Go to dashboard and change order status to "Refund".
  7. Now go to Payments.
  8. Click on Transactions option.
  9. Observe that Refunded transactions fails to appear

Actual Result:
Refunded transactions fails to appear in Payments -> Transaction tab.

Expected Result:
Refunded transactions should appear on the Payments -> Transaction tab

#26563.zip

@dechov
Copy link
Contributor

dechov commented May 20, 2020

Thanks for bringing it to our attention.

Is there an expectation that changing the order status would be equivalent to a full refund using the "Refund $x.xx via WooCommerce Payments" action?

@juliaamosova
Copy link
Author

juliaamosova commented May 20, 2020

@dechov thanks! I took a look at the video (at the end of the issue) and then logged in to the site where it was tested (on Jurassic Ninja) by the 3rd company that we contracted for testing. I see that the order was refunded from within WooCommerce (not just order status was changed) but I also see a note that order needs to be refunded through payment gateway as well:

wc_pay

Can you please clarify? Do the merchants need to login to Stripe to issue a refund? I am looking through docs now as well.

@juliaamosova
Copy link
Author

Can you please clarify? Do the merchants need to login to Stripe to issue a refund? I am looking through docs now as well.

I see that logging in to Stripe is not needed as per our docs:

https://docs.woocommerce.com/document/payments/#section-8

@juliaamosova
Copy link
Author

Ok, I see what happened. They just changed the order status from Processing to Refunded and the amount got refunded without actually refunding it - expected behavior of WooCommerce.

I just tried the same on another order on the same test site as where the issue was tested and filed and this is indeed what happens after changing the order status from Completed to Refunded:

Screen Shot 2020-05-20 at 3 26 15 PM

I also tried to process refund via Refund button as it should be but couldn't (see order notes on the screenshot above). I am not the one who set up this site and payments on it so hard to tell right away what is happening but it is not related to the original issue described in this thread.

Now to your question:

Is there an expectation that changing the order status would be equivalent to a full refund using the "Refund $x.xx via WooCommerce Payments" action?

I don't think this is an expectation, no. The refund should be done via Refund button. But it is indeed confusing that we allow for the order to be refunded visually when the order status is simply changed to Refunded. But this is a question for WooCommerce Core.

Feel free to close this issue unless you think otherwise.

@dechov
Copy link
Contributor

dechov commented May 20, 2020

That screenshot is what I get when switching the status to "Refunded". It's effectively the same (except for the order note) as the manual refund.

Currently, the only way to issue a refund is with the automatic refund button:

We're aware that the manual vs. automatic refund distinction could be confusing (see #426), particularly in this context of an the integrated gateway, but haven't found a solution yet, and also weren't considering the status change as another aspect of it.

@juliaamosova
Copy link
Author

juliaamosova commented May 20, 2020

That screenshot is what I get when switching the status to "Refunded". It's effectively the same (except for the order note) as the manual refund.

Yep, noted, thanks! Getting the same.

Currently, the only way to issue a refund is with the automatic refund button:

Thanks for confirming! Noted.

We're aware that the manual vs. automatic refund distinction could be confusing (see #426), particularly in this context of an the integrated gateway, but haven't found a solution yet, and also weren't considering the status change as another aspect of it.

Thanks! As far as changing the order status to Refunded: current behavior is how it works in Core. So likely, not something that can be easily fixed on the payment side of things. Also likely a bigger discussion is to be had around this.

We can close this issue for now. I will reply to the author via the issue they opened in the core's repo: woocommerce/woocommerce#26563.

Thanks for taking a look so quickly @dechov!

@allendav
Copy link
Contributor

Thanks both of you. Closing as not-an-issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants