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

Failed orders should include more info on transaction details page #5713

Closed
aheckler opened this issue Mar 9, 2023 · 7 comments · Fixed by #9980
Closed

Failed orders should include more info on transaction details page #5713

aheckler opened this issue Mar 9, 2023 · 7 comments · Fixed by #9980
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. component: timeline and events Issues related to Timeline and Events focus: payments acceptance & processing

Comments

@aheckler
Copy link
Member

aheckler commented Mar 9, 2023

If an order fails for some reason, say insufficient funds, there's a lack of information about that on the transaction details page. Here's an example screenshot:

Screenshot taken on 2023-03-09 at 19 43 44 UTC@2x

You can see there that this order, although it failed, has two bits of info present:

  • An order number
  • A failure/decline reason

Here's the corresponding transaction page under Payments > Transactions:

Screenshot taken on 2023-03-09 at 19 45 03 UTC@2x

Not only is the order number missing, but the decline reason is nowhere to be seen.

In the interest of giving our merchants as much info as we can, IMHO we should include this in the transaction detail page.

This idea stems from this post: paJDYF-7eg-p2

@aheckler aheckler added type: bug The issue is a confirmed bug. needs triage Manually put issue into triage process. labels Mar 9, 2023
@htdat htdat added component: timeline and events Issues related to Timeline and Events category: core WC Payments core related issues, where it’s obvious. needs prioritisation Triage finished and issues are ready for the following processing. and removed needs triage Manually put issue into triage process. type: bug The issue is a confirmed bug. labels Mar 10, 2023
@jessy-p
Copy link
Contributor

jessy-p commented Jan 23, 2024

This issue impacts Transactions (Auth and Capture), so assigning to Pulsar (based on team responsibilities Pc2DNy-3z-p2) @kalessil. Assigning as part of Gamma Triage process PcreKM-yM-p2.

Tagging as part of re-evaluating older issues in the backlog, please have a look and close if no longer relevant.

@kalessil kalessil removed the needs prioritisation Triage finished and issues are ready for the following processing. label Feb 19, 2024
@kalessil
Copy link
Contributor

@mgascam : please re-evaluate.

@mgascam
Copy link
Contributor

mgascam commented Dec 17, 2024

Hi @aheckler, I know some time has passed since you reported this issue, but I was wondering if you could share the steps to reproduce a failed order like the one you described. I'm struggling to recreate it on my end.

For example, when I use a test card that fails due to insufficient funds, I can't get past the checkout step. The order transitions to the "Failed" status, but no payment intent is created on Stripe's end, so I can't access the payment details view.

For context, I'm working on a branch that addresses this issue. It includes unit tests, but I’d like to add testing instructions to verify this flow manually. Unfortunately, I haven’t been able to find a way to trigger it yet, which is why I’m reaching out.

Thanks in advance for your help!

@mgascam mgascam linked a pull request Dec 17, 2024 that will close this issue
6 tasks
@aheckler
Copy link
Member Author

@mgascam I tried a couple different ways but I don't see how to replicate that either. :/ @AashikP or @francoishvz may know of a way?

@AashikP
Copy link

AashikP commented Dec 18, 2024

I could not replicate this in test mode

Image

@AashikP
Copy link

AashikP commented Dec 18, 2024

I've seen a conflict in most cases where the order number is not linked. I just realized we also have this latest example for WooPay: p1734327561455129-slack-C7U3Y3VMY

cc @nicdwilson @reykjalin

@mgascam
Copy link
Contributor

mgascam commented Dec 18, 2024

Thanks for attempting to reproduce the issue, @aheckler and @AashikP. Other folks in Gamma also couldn't reproduce it. Since this issue was reported quite some time ago, it's possible that the flow has changed, making it harder to replicate, or it could be related to testing mode.

Regardless, we are now including the failure reason in the timeline after this PR is merged. This will show the failure reason provided by Stripe, which was previously missing. I expect this change will help clarify why transactions fail.

Regarding the order, I didn’t include any changes because we already include it—as long as the order metadata is linked to the payment intent in Stripe, regardless of whether the payment intent fails or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. component: timeline and events Issues related to Timeline and Events focus: payments acceptance & processing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants