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

Allow reconciliation of bank statements & daily deposits - e.g. show deposit Payout trace ID in Woo admin dashboard, or other unique ID to link #5316

Closed
nicdwilson opened this issue Dec 20, 2022 · 33 comments · Fixed by #9832
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. focus: payouts priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: enhancement The issue is a request for an enhancement.

Comments

@nicdwilson
Copy link

nicdwilson commented Dec 20, 2022

Description

In 5789782-zen a new merchant with a fairly busy site has asked if there is any way to provide additional information to allow easy reconciliation of bank statements to deposits.

The bank statement contains the Payout trace ID.

The Payout trace ID is not available in reporting on the site, nor is it available via the payout.paid event. But would be quite helpful for merchants' accounts departments trying to match up daily deposits with their bank statements, especially considering that pending periods sometimes make matching by date alone tricky.

Additional information ( added by @nagpai )

@jessy-p jessy-p added category: core WC Payments core related issues, where it’s obvious. type: enhancement The issue is a request for an enhancement. labels Dec 30, 2022
@jessy-p
Copy link
Contributor

jessy-p commented Feb 2, 2023

@Automattic/helix

@haszari haszari changed the title Additional reporting data to allow easy reconciliation of daily payments and bank statements Show deposit Payout trace ID in WCPay UI to allow reconciliation of bank statements & daily deposits Feb 12, 2023
@haszari
Copy link
Contributor

haszari commented Feb 12, 2023

If we can get this ID from the API, seems like a great idea to include in the UI somewhere. Or any other value that would help clearly identify WCPay deposits (in general, or specific deposit instances).

I couldn't find mention of it in the Stripe docs, except Transfer/payout ID in Balance report.

FYI Sourav – let's consider this in upcoming deposits project(s).

@haszari haszari added the component: payouts Issues related to Payouts label Feb 12, 2023
@souravdebnath1986
Copy link

souravdebnath1986 commented Feb 12, 2023

This falls in the reporting enhancements bucket for this year.

@haszari
Copy link
Contributor

haszari commented Feb 13, 2023

Thanks @souravdebnath1986 !

This falls in the reporting L2 bucket.
Provide basic reporting capabilities for merchants to support discovery, settlement reconciliation and delivery preferences

Since we're planning feature projects in this area later in the year, perhaps we can close this issue.

@haszari haszari added the category: projects For any issues which are part of any project, including bugs, enhancements, etc. label Feb 14, 2023
@haszari
Copy link
Contributor

haszari commented Feb 14, 2023

Removing from maintenance board & adding projects label based on @souravdebnath1986 comment above.

There's a possibility that adding this payout ID in the UI will help make the deposits UI & flow easier to merchants to understand (i.e. inspire trust). If so we could consider this for upcoming deposits UI refresh project.

@brucealdridge
Copy link
Contributor

@Automattic/fusion This issue mentions deposits but falls under the future reporting project. Can you consider it as part of your roadmap?

@brucealdridge brucealdridge added the component: reporting Issues related to Reporting label Oct 18, 2023
@anu-rock
Copy link
Contributor

@brucealdridge Thanks for the ping. Are we talking about the deposits CSV report here?

@brucealdridge
Copy link
Contributor

Are we talking about the deposits CSV report here?

@anu-rock The issue specifically mentions the UI. I'd imagine it could also appear in the deposits CSV report.

@anu-rock
Copy link
Contributor

anu-rock commented Nov 2, 2023

@brucealdridge Thanks for confirming. Added it to Fusion's backlog.

@haszari haszari removed the component: payouts Issues related to Payouts label Nov 21, 2023
@haszari
Copy link
Contributor

haszari commented Nov 21, 2023

Removed deposits label since this will be handled by Fusion as part of reporting.

@jtobiesen
Copy link

jtobiesen commented Nov 23, 2023

Another request in 7340232-zd-a8c.
This merchant specifically requested to have information about the order numbers included in each associated deposit, somehow reflected in the payout.

@nicdwilson
Copy link
Author

7311071-zd-a8c

@csmcneill
Copy link
Contributor

If we can get this ID from the API, seems like a great idea to include in the UI somewhere.

Stripe has indicated that they will be surfacing the trace ID via the API in 1H2024. They'll include us in the beta.

@csmcneill csmcneill added the status: blocked The issue is blocked from progressing, waiting for another piece of work to be done. label Nov 27, 2023
@anu-rock anu-rock added the priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability label Dec 7, 2023
@souravdebnath1986
Copy link

souravdebnath1986 commented Dec 11, 2023

We are picking this up as part of p7fD6U-b70-p2 to include a deposit_id in the Transaction Report. We will also do a fast follow to update this id as part of the Deposit Report to aid reconciliation of Deposit to cash as seen in the bank statement. cc: @anu-rock

@mordeth mordeth self-assigned this Dec 12, 2023
@haszari haszari changed the title Show deposit Payout trace ID in WCPay UI to allow reconciliation of bank statements & daily deposits Allow reconciliation of bank statements & daily deposits - e.g. show deposit Payout trace ID in Woo admin dashboard, or other unique ID to link Dec 15, 2023
@haszari
Copy link
Contributor

haszari commented Dec 15, 2023

The ideal solution to this would include a deposit ID of some kind (e.g. payout trace ID) in both the Woo admin dashboard AND the merchant's bank statement.

@mordeth @anu-rock Could you investigate that as part of your effort on this issue? If it's challenging or you hit a blocker, feel free to reach out to myself or @souravdebnath1986 , we can add a follow up issue if necessary.

@SiR-DanieL
Copy link
Member

Merchant here from A8C, I also have this issue with the Trade ID missing from WC Pay and it makes my life a nightmare. An extensive discussion about the issue is in Slack: p1702963566265379-slack-C3NCP7ZJ6

@souravdebnath1986 souravdebnath1986 added priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. and removed priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability labels Dec 20, 2023
@lynnjat7
Copy link

Another report of this causing accounting issues for EU customers: 8779859-zen

@carlcarvesstone
Copy link

carlcarvesstone commented Sep 25, 2024

Good Folks,

Small businesses, or non-profits, who do not have an accounting team (our non-profit supports stone sculptors), often pay for an actual accountant or bookkeeper outside the organization to do the bank reconciliations, for reasons of accuracy, trust, and transparency (and more likely the lack of accounting skills within the non-profit).

The lack of a deposit ID/REF causes some additional amount of outside fees to be paid as a work around to not having this feature. I see it as a hidden cost of using your product. For the small non-profit, for which I volunteer, our accountant/bookkeeper estimated, given our range of transaction activity, $300-$800 per year to do the deposit reconciliation work for our organization. This is an additional cost to our non-profit due to the lack of this feature.

If the deposit ID feature were implemented the level of effort for reconciliation is close to zero. Please see this as closing the loop on customer satisfaction and removing your customer's need to report to their board/treasurer why the Woo package is causing additional expense, and making it expensive to have a clean audit. I felt pretty embarrassed about explaining my choice of Woo and the additional costs, and of course gives me pause in recommending Woo for small organizations.

I have no idea the number of small organizations like ours that use Woo, so I have no way to judge benefit and multiplier effect of getting this done (our previous web site software ID/refs were provided).

Looking forward to a solution.
Thanks

@haszari
Copy link
Contributor

haszari commented Nov 12, 2024

This enhancement was blocked because we (WooPayments) didn't have access to the Payout Trace ID in the Stripe API … however …

Stripe has indicated that they will be surfacing the trace ID via the API in 1H2024. They'll include us in the beta.

Looks like this is now documented as a preview feature in the Stripe API:

Image

So I'll remove blocked, we can investigate and see if this is available.

@haszari haszari removed the status: blocked The issue is blocked from progressing, waiting for another piece of work to be done. label Nov 12, 2024
@haszari
Copy link
Contributor

haszari commented Nov 14, 2024

Bumping this to high, it's important and 🤞🏻 hopefully feasible!

@haszari haszari added priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. focus: payouts and removed priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. component: reporting Issues related to Reporting focus: reporting labels Nov 14, 2024
@nagpai nagpai self-assigned this Nov 15, 2024
@nagpai
Copy link
Contributor

nagpai commented Nov 15, 2024

I did a quick check of a recent payouts for a few accounts and found that the Trace ID is still not exposed via the payout.paid webhook.

In a few cases I do see the trace_id object, but it's status is stuck at pending with value nil

I have followed up with Stripe on p1731688692196839-slack-C9976E5MJ to see if there is any change, or anything that they need to enabled for us.

@haszari
Copy link
Contributor

haszari commented Nov 17, 2024

Looks like this is now documented as a preview feature in the Stripe API

I did a quick check of a recent payouts for a few accounts and found that the Trace ID is still not exposed via the payout.paid webhook.

I have followed up with Stripe on p1731688692196839-slack-C9976E5MJ to see if there is any change, or anything that they need to enabled for us.

Note from that thread – trace_id is only available for US and CA merchant accounts. So if we implement it will not be displayed for all merchants – inconsistent experience.

@carlcarvesstone
Copy link

carlcarvesstone commented Nov 18, 2024 via email

@nagpai
Copy link
Contributor

nagpai commented Nov 18, 2024

Note from that thread – trace_id is only available for US and CA merchant accounts. So if we implement it will not be displayed for all merchants – inconsistent experience.

The trace_id is shown with status as pending and value as null even for US accounts. I have asked for a further check -p1731906278347669/1731688692.196839-slack-C9976E5MJ

@nagpai nagpai added the status: blocked The issue is blocked from progressing, waiting for another piece of work to be done. label Nov 19, 2024
@nagpai
Copy link
Contributor

nagpai commented Nov 19, 2024

Marking as blocked, since we are awaiting a reply from Stripe

The trace_id is shown with status as pending and value as null even for US accounts. I have asked for a further check -p1731906278347669/1731688692.196839-slack-C9976E5MJ

@nagpai nagpai removed the status: blocked The issue is blocked from progressing, waiting for another piece of work to be done. label Nov 26, 2024
@nagpai
Copy link
Contributor

nagpai commented Nov 26, 2024

Stripe has replied confirming the trace_id is now available for payouts ( from November 21, 2024 )

I did a quick check and found it visible on transactions on a US, UK and a German account too!

@nagpai
Copy link
Contributor

nagpai commented Dec 2, 2024

@carlcarvesstone , @SiR-DanieL - Happy to share that a Bank reference key will be available within the payout reports ( view and CSV export ). This will help you reconcile your bank statements! This will be released in WooPayments 8.6.0 within the next few days.

@nagpai
Copy link
Contributor

nagpai commented Dec 2, 2024

Note from that thread – trace_id is only available for US and CA merchant accounts. So if we implement it will not be displayed for all merchants – inconsistent experience.

@haszari - At the time of checking, I was able to see this for all countries, which is good! There is a fallback of N/A in case there is no trace_id in the Stripe webhook.

@nagpai
Copy link
Contributor

nagpai commented Dec 9, 2024

Reopening issue for a minor change in the label Bank reference key to Bank reference ID based on feedback from @aheckler . Ref: p1733483582646479/1732857021.096589-slack-C02BW3Z8SHK

@nagpai
Copy link
Contributor

nagpai commented Dec 10, 2024

Closing the original issue here and moving the renaming task to its own issue - #9909

@nagpai nagpai closed this as completed Dec 10, 2024
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. focus: payouts priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: enhancement The issue is a request for an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

15 participants