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
Add refund reports #5791
Add refund reports #5791
Conversation
Unlike #5783, this one actually worked for me in dev enviroment. @NicolasDorier Thoughts on giving crypto amount rate and currency? IMO these should reflect the typical payment report just do it for refunds and provide as much info as possible, you never know what people actually need from them. Can you explain what fully paid status is? Is it full |
@pavlenex Fully Paid is saying if all the money has been transferred from the pull payment of the refund. That said, for BTCPay 2.0, we can start saving additional information for future refunds. |
62b5441
to
d4afe0e
Compare
This is now ready and tested. I changed the name of the columns to reflect what is used in the pull payments page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, but I have't tested in-depth.
1ec451c
to
6602f7e
Compare
Refund reports.
I didn't see @Kukks has been working on it, possible alternative to #5783
Reviewed Kukks's one, outside a minor point (not using the display formatter), it's also good, no strong preferences.
Still need tests