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

"Debit failed for (no description)" #1573

Closed
msherry opened this issue Dec 12, 2014 · 8 comments
Closed

"Debit failed for (no description)" #1573

msherry opened this issue Dec 12, 2014 · 8 comments

Comments

@msherry
Copy link

msherry commented Dec 12, 2014

image

https://dp.balancedpayments.com/#/marketplaces/MP17rGQI6yMEysofZfzYkpNJ/customers/CU40IVXOldo9FfPSupmGjtzh

@kyungmin
Copy link
Contributor

@msherry This is an intended behavior. These debits don't have descriptions. Feel free to reopen if you have a specific suggestions.

@msherry msherry reopened this Dec 12, 2014
@msherry
Copy link
Author

msherry commented Dec 12, 2014

@kyungmin Those transactions have failure reasons. Click on the first one: "477: This transaction was declined. Please contact the card processor for more information about the failure reason."

I see that there's a new field called "Internal description", which is indeed none. I don't know where that comes from or how it's populated, but that seems like a mistake, since immediately above it is a legit failure reason.

image

@kyungmin
Copy link
Contributor

Ah, now I understand what you were confused about. Sounds like "for" is the confusing point.

"Internal description" is typically how the dashboard users identify transactions, and that's why we brought back the description as "Debit succeeded for Booking 123" (#1565 (comment)).

I think something like "Debit (Booking 123) succeeded" or "Debit (no description) failed" might be more clear.

@msherry
Copy link
Author

msherry commented Dec 12, 2014

Ahh yup, that's it. 'For' implies a reason to me.

I like "Debit (Booking 123) succeeded", but if there's no description, why include anything at all? "Debit failed" seems like it conveys the same amount of information in less space.

@kyungmin
Copy link
Contributor

Yes, that would be a more elegant solution.

@kyungmin
Copy link
Contributor

The problem with "Debit (Booking 123) succeeded" is that "succeeded" could get cut off in the table if the description is long, but we will go with this solution until we discover a better one.

@mjallday
Copy link
Contributor

you've got the red denoting failure tho?

@kyungmin
Copy link
Contributor

Right, that's why I think it's not a huge problem. All transactions with descriptions have simple statuses, like succeeded/pending/failed, which can be easily assumed from the colored dot.

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